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

Flower Datasets Quickstart typo: MNIST instead of CIFAR10 #2808

Merged
merged 2 commits into from
Feb 16, 2024

Conversation

gubertoli
Copy link
Contributor

Issue

The quickstart tutorial of Flower Datasets has a minor typo in the description, mentioning MNIST instead of CIFAR10.

Description

This PR only changes the description from MNIST to CIFAR.

Related issues/PRs

N/A

Proposal

From:
Now you’re ready to go. You have ten partitions created from the train split of the MNIST dataset...

To:
Now you’re ready to go. You have ten partitions created from the train split of the CIFAR10 dataset...

Explanation

N/A

Checklist

  • Implement proposed change
  • Write tests
  • Update documentation
  • Update the changelog entry below
  • Make CI checks pass
  • Ping maintainers on Slack (channel #contributions)

Changelog entry

Any other comments?

N/A

@helin0815
Copy link
Contributor

Great!

@adam-narozniak
Copy link
Contributor

Good catch!

@tanertopal tanertopal enabled auto-merge (squash) February 16, 2024 16:35
Copy link
Member

@tanertopal tanertopal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the fix!

@tanertopal tanertopal merged commit 6da77b1 into adap:main Feb 16, 2024
30 checks passed
@gubertoli gubertoli deleted the minor-flwr-dataset-typo branch June 17, 2024 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants