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

feat(baselines): Add FedLC #4548

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from
Draft

feat(baselines): Add FedLC #4548

wants to merge 9 commits into from

Conversation

flydump
Copy link

@flydump flydump commented Nov 19, 2024

Issue

Description

Adds FedLC (Federated Learning with Label Distribution Skew via Logits Calibration) baseline

Related issues/PRs

Proposal

Explanation

Merge baseline

Checklist

  • Implement proposed change
  • Write tests
  • Update documentation
  • Make CI checks pass
  • Ping maintainers on Slack (channel #contributions)

Any other comments?

This PR is still very much still work in progress. I have found it hard to reproduce the results from the paper due to lack of experimental details.

@jafermarq
Copy link
Contributor

Hello @flydump 👋 , great to see a PR open for FedLC. Let us know when you'd like us to do a first review. Also, do not hesitate to ask questions if you have some doubts on how to implement certain aspects of the FedLC logic using Flower. Happy to help!!

@flydump
Copy link
Author

flydump commented Dec 9, 2024

Hello @flydump 👋 , great to see a PR open for FedLC. Let us know when you'd like us to do a first review. Also, do not hesitate to ask questions if you have some doubts on how to implement certain aspects of the FedLC logic using Flower. Happy to help!!

Thanks, unfortunately I haven't been able to reproduce the results of the paper reliably despite my best efforts (I have compiled my latest results in the README if you'd like to take a look). So I'm probably going to pause further development on this baseline.

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

Successfully merging this pull request may close these issues.

2 participants