Update on Federated Analytics example #2896
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue
Description
This PR proposes a change to the former Federated Analytics example. Previously, the Federated Analytics example was tied to the use of Pandas, and the example's name was
quickstart-pandas
.Later changes loaded the Iris dataset with Flower Datasets with the method
.with_format("pandas")
to keep aligned with thequickstart-pandas
.For a matter of context, I recently had some issues during my search for this Federated Analytics example because of the misleading name as
quickstart-pandas
.Then, my PR aims to clear this by keeping the example only within the Federated Analytics scope.
Overall, my understanding is that it is also beneficial for Flower positioning with this clear example showing the framework as applicable for federated analytics applications.
Another example with Pandas (for loading external/custom data not available on Hugging Face and Flower Datasets) can be created as a new quickstart (I can do that).
Related issues/PRs
Initial PR is #1469 and later changed by #2816.
Proposal
Explanation
README.md
with a more detailed explanation, and rearranging the sequence of execution steps.client.py
to do not use pandas anymorerequirements.txt
andpyproject.toml
to remove pandas dependency.docs
files to keep aligned with the changesChecklist
#contributions
)Changelog entry
Any other comments?
N/A