NVIDIA / NVFlare

NVIDIA Federated Learning Application Runtime Environment
https://nvidia.github.io/NVFlare/
Apache License 2.0
592 stars 165 forks source link

Fix Fed Stats due to recent changes #2691

Closed chesterxgchen closed 1 month ago

chesterxgchen commented 1 month ago
  1. fed_stats is broken due to the simulator changes, change the workspace directory in the notebooks and examples
  2. for df_stats job, notice the new datasets has empty line at end, causing pandas to mistaken numerical data type to string type. We change the prepare_data.py to remove empty line
  3. update df_stats notebook to add panda data table format
  4. cleaned notebooks output

Types of changes

chesterxgchen commented 1 month ago

/buld

chesterxgchen commented 1 month ago

/build

chesterxgchen commented 1 month ago

LGTM. Are there any changes needed in https://github.com/NVIDIA/NVFlare/blob/main/examples/hello-world/step-by-step/higgs/stats/tabular_stats.ipynb?

good point, I need to fix that as well. Let me merge this and fix that in separate PR