Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Byzantine robust aggregator #597

Open
s314cy opened this issue Sep 11, 2023 · 2 comments
Open

Byzantine robust aggregator #597

s314cy opened this issue Sep 11, 2023 · 2 comments
Labels
bug Something isn't working discojs Related to Disco.js server Related to the server
Milestone

Comments

@s314cy
Copy link
Contributor

s314cy commented Sep 11, 2023

Following the aggregator refactor in #578, byzantine-robust aggregation in the federated setting does not work anymore as it remains to be re-implemented with the new architecture in mind.

@s314cy s314cy added bug Something isn't working discojs Related to Disco.js server Related to the server labels Sep 11, 2023
@JulienVig
Copy link
Collaborator

The work pre-refactoring has been left in the code unused, we are removing it in #652. For future use, it can be found in discojs/discojs-core/src/aggregator/robust.ts at commit b9db50f

@JulienVig JulienVig added this to the v4.0.0 milestone Jul 23, 2024
@tharvik
Copy link
Collaborator

tharvik commented Jul 23, 2024

thanks @JulienVig for the code de-zombification

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working discojs Related to Disco.js server Related to the server
Projects
None yet
Development

No branches or pull requests

3 participants