Add FL example with nvflare #189
Conversation
Signed-off-by: Yiheng Wang <vennw@nvidia.com>
yiheng-wang-nv
changed the title
Signed-off-by: Yiheng Wang <vennw@nvidia.com>
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
yiheng-wang-nv
changed the title
Signed-off-by: Yiheng Wang <vennw@nvidia.com>
Signed-off-by: Yiheng Wang <vennw@nvidia.com>
|
will this break our daily notebook tests? https://github.com/Project-MONAI/MONAI/runs/2644488450?check_suite_focus=true#step:6:11 the notebooks should run in the right order, could you rename the notebooks into "1-startup", "2-admin", "3-server", "4-client"? |
Signed-off-by: Yiheng Wang <vennw@nvidia.com>
Hi @wyli , I modified the notebook names. |
federated_learning/nvflare/spleen_example/config/dataset_part1.json
Outdated
Show resolved
Hide resolved
federated_learning/nvflare/spleen_example/config/dataset_part2.json
Outdated
Show resolved
Hide resolved
federated_learning/nvflare/spleen_example/config/config_fed_server.json
Outdated
Show resolved
Hide resolved
Signed-off-by: Yiheng Wang <vennw@nvidia.com>
Signed-off-by: Wenqi Li <wenqil@nvidia.com>
Signed-off-by: Wenqi Li <wenqil@nvidia.com>
…tiple nodes) Signed-off-by: Wenqi Li <wenqil@nvidia.com>
Signed-off-by: Wenqi Li <wenqil@nvidia.com>
| @@ -50,7 +50,7 @@ doChecks=true | |||
| doRun=true | |||
| autofix=false | |||
| failfast=false | |||
| pattern="-name \"*\"" | |||
| pattern="-and -name '*' -and ! -wholename '*federated_learning*'" | |||
There was a problem hiding this comment.
Hi @rijobro I exclude the federated_learning folder by default, because it will likely to use multiple node, which is not feasible for the CI/CD at the moment.
There was a problem hiding this comment.
ok, although I think the change should have really gone into our CI/CD .yaml files, rather than the runner itself.
* Add flare based trainers Signed-off-by: Yiheng Wang <vennw@nvidia.com> * Refactor the whole pipeline * Rename fl example * Update nvflare example Signed-off-by: Yiheng Wang <vennw@nvidia.com> * Fix pep8 errors Signed-off-by: Yiheng Wang <vennw@nvidia.com> * Specify nvflare version Signed-off-by: Yiheng Wang <vennw@nvidia.com> * Update copyrights and docker info Signed-off-by: Yiheng Wang <vennw@nvidia.com> * Use fixed monai docker version and add new lines Signed-off-by: Yiheng Wang <vennw@nvidia.com> * fixes readme Signed-off-by: Wenqi Li <wenqil@nvidia.com> * update readme Signed-off-by: Wenqi Li <wenqil@nvidia.com> * by default exclude the federated learning related (often requires multiple nodes) Signed-off-by: Wenqi Li <wenqil@nvidia.com> * fixes a link Signed-off-by: Wenqi Li <wenqil@nvidia.com> Co-authored-by: Wenqi Li <wenqil@nvidia.com>
Signed-off-by: Yiheng Wang vennw@nvidia.com
Description
This PR shows an example of using a MONAI trainer and nvflare to do federated learning.
Status
Ready