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

[Please Don't Merge] implement a new ModeBasedAssigner for FedEval #1226

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

ishaileshpant
Copy link
Collaborator

@ishaileshpant ishaileshpant commented Dec 23, 2024

  • implement a new ModeBasedAssigner class extending RandomGroupedAssigner

  • update defaults/assigner.yaml to use ModeBasedAssigner with default mode to train_and_validate

  • update fedeval sample workspace to use default assigner, tasks and aggregator

  • Testcases for ModeBasedAssigner

  • Testing for all existing workspaces

  • Change all workspaces to use new Assigner

  • Documentation update

@ishaileshpant ishaileshpant added this to the 1.8 milestone Dec 23, 2024
@ishaileshpant ishaileshpant force-pushed the poc-mode-assigner branch 18 times, most recently from 3020198 to f1fcdcf Compare December 24, 2024 11:12
…gner

- update defaults/assigner.yaml to use ModeBasedAssigner with default mode to train_and_validate
- update fedeval sample workspace to use default assigner, tasks and aggregator
- use of federated-evaluation/aggregator.yaml for FedEval specific workspace example to use round_number as 1
- removed assigner and tasks yaml from defaults/federated-evaluation, superseded by default assigner/tasks
- add happy flow tests for ModeBasedAssigner
- add negative tests for ModeBasedAssigner
- TODO revert workflow change post validation on workflow run in draft mode
Signed-off-by: Shailesh Pant <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant