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

Merge FL and MIA handlers #127

Open
johanos1 opened this issue Sep 3, 2024 · 0 comments
Open

Merge FL and MIA handlers #127

johanos1 opened this issue Sep 3, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@johanos1
Copy link
Collaborator

johanos1 commented Sep 3, 2024

Feature

Desired Behavior / Functionality

We should only have a single handler for MIA and FL

What Needs to Be Done

  1. Update metaoptimizers to maintain the patched model parameters
  2. Update abstract handler to check for mia or gia to create patched model
  3. Make user implement _train()
  4. Force _train() to accept an optimizer the same way for MIA and GIA.

How Can It Be Tested

Add FL handler to unit test

@johanos1 johanos1 added the enhancement New feature or request label Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant