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

Missing weights when averaging #1

Open
omarfoq opened this issue Jan 15, 2021 · 0 comments
Open

Missing weights when averaging #1

omarfoq opened this issue Jan 15, 2021 · 0 comments

Comments

@omarfoq
Copy link

omarfoq commented Jan 15, 2021

Hello,

I am not sure if this an error, but it seems you are not weighting the clients when averaging reduce_add_average. Moreover, in the paper you use Local SGD with same number of steps at each client, while in the code you fit a fixed number of epochs, thus the number of steps is not constant across clients, and depends on the local data size.

I am not sure if what I highlight here is relevant, can you please clarify this if possible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant