-
Notifications
You must be signed in to change notification settings - Fork 941
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
feat(baselines) Add baseline FedBABU #4322
base: main
Are you sure you want to change the base?
Conversation
Hi, @jafermarq For example, I need |
Hi @KarhouTam, what version of flower I you using? The flower baselines template is using the latest release as a minimum version:
Note to create the baseline, it's best to do it like this (since it will also populate a
|
Seems I forgot to update my local flwr version first 😂. Nevermind. |
33d804d
to
96222d6
Compare
Hi @KarhouTam, Just checking in here. Seems like the implementation is nearly passing all tests. Are there any issues you are facing? |
Thanks, @WilliamLindskog . Everything is fine so far. I am currently busy on other things so this PR is panding for now. Once I am free, the process will be re-launched. I will ask for help if I meet problems in that time. |
Hi @KarhouTam, Just checking in here to see if there's anything we can support you with. |
Nah. Just forgot about this thread 😂. Flower framework is friendly to reproduction. So I might be re-launch this later? |
Sure thing, I might close related issue if this is already being worked on and migrate this baseline to Flower Discuss if someone wants to pick this up before we can merge it. |
@jafermarq seems like most tests pass but some baseline stuff doesn't run, have you seen this before? One error doesn't really show an output log. @KarhouTam did you encounter an error when running locally? |
Actually the algorithm code is not finished yet. I'm pushing the process locally and will update this thread when most things are done. |
Issue
Add new baseline FedBABU
https://arxiv.org/abs/2106.06042
Description
Related issues/PRs
Placeholder issue: #4285