FL4Health icon indicating copy to clipboard operation
FL4Health copied to clipboard

Changes to Support Expanded Experimentation with FedDG-GA

Open emersodb opened this issue 1 year ago • 0 comments

PR Type

Feature/Experimentation.

Short Description

The changes in this PR are targeted at enabling expanded experimentation with the FedDG-GA strategy for a wider set of FL approaches. Also included is a name change, FedDgGaStrategy -> FedDgGa to fit other strategy formats a bit better and renaming of the associated file from feddg_ga_strategy.py to feddg_ga.py as it is already under the strategies folder.

For FENDA+Ditto, there is also a bug fix moving from SequentiallySplitExchangeBaseModel to SequentiallySplitModel. In this setting we want to exchange the whole Ditto model, not just the feature extractor component. It wasn't causing a real bug, as a FullParameterExchanger was being used anyway, but the typing was dissonant.

Finally, I moved some "client agnostic" functionality out of basic client and into a utils file to help trim a few functions from the BasicClient class.

Tests Added

Added a test for the new Fed DG-GA strategy that is compatible with adaptive constraint server-client pairs

emersodb avatar Oct 10 '24 14:10 emersodb