-
Notifications
You must be signed in to change notification settings - Fork 906
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
Fedmeta #2438
Fedmeta #2438
Conversation
…s removed from pool
Co-authored-by: Daniel J. Beutel <[email protected]>
Co-authored-by: Daniel J. Beutel <[email protected]>
@jafermarq, I have completed the list you suggested. Please let me know what further steps I need to take. |
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.
@jinsoogod , thanks for making those changes. You'll notice removed some files that were not part of your baseline/fedmeta
(there are a couple more that i need to look into -- not sure what's going on exactly).
Could you in the meantime:
- Run the
format-baseline.sh
script again? some things need to change (this will happen automatically) - make the suggested changes to the readme.
Thank you.
Co-authored-by: Javier <[email protected]>
@jafermarq , I've fixed your edits now. Please check |
@jinsoogod, I have added some updates to fix the formatting and also discard files that weren't part of the baseline. Until now i've only tested your code with Shakespeare (which i'm running now for all 400 rounds). I'm running FEMNIST for the first time and I noticed some errors in the
Could you:
|
@jafermarq, You are right. LEAF github was created with under pillow version 9.5. I downloaded this data from the existing LEAF Anaconda environment. The current error occurs during the download process. This does not occur during preprocessing that maintains raw data. My Pillow version is 9.5.0 |
Ok, i'll add that as part of |
Thank you for your hard work. I'm waiting for good news. |
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.
Looks great! @jinsoogod
Description
Implements FedMeta baseline.
Related issues/PRs
See issue 2241
Checklist
#contributions
)