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

Fix client state error #291

Merged
merged 1 commit into from
Aug 8, 2022
Merged

Fix client state error #291

merged 1 commit into from
Aug 8, 2022

Conversation

joneswong
Copy link
Collaborator

Some changes have not been reflected in subclasses of server, as #289 reported.

@joneswong joneswong added the bug Something isn't working label Aug 8, 2022
@joneswong joneswong requested a review from xieyxclack August 8, 2022 03:01
Copy link
Collaborator

@xieyxclack xieyxclack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@xieyxclack xieyxclack merged commit 44ac3fd into alibaba:master Aug 8, 2022
@joneswong joneswong deleted the fix_fedex branch August 9, 2022 06:35
Schichael pushed a commit to Schichael/FederatedScope_thesis that referenced this pull request Sep 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug appears when the elements in a Sampler's attribute self.client_state are all zeros
2 participants