-
Notifications
You must be signed in to change notification settings - Fork 1
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
added past code #1
base: main
Are you sure you want to change the base?
Conversation
PRs are the best way to review code instead of pushing directly. the branch can live in your fork. |
await load_weights(client_id) | ||
|
||
async def fed_avg(): | ||
await load_data() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
await load_data() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can load the weights in the for loop below.
self.assertAllClose(model.model.trainable, | ||
iterative_process.get_model_weights(model).trainable) | ||
|
||
for _ in range(num_clients): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
load the weights for the latest submitted model from each client inside this loop.
i think i implemented most of the changes from |
i tried pushing a branch directly to neuronets but it says access denied, would it be preferable to still use branches in the future?