Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 12, 2026

The CLI exercise documentation references training on the iris dataset, but all exercise files load the breast cancer dataset instead.

Changes

  • Updated s2_organisation_and_version_control/cli.md line 195 to say "breast cancer dataset" instead of "iris dataset" to match the actual code implementation

The documentation now correctly states:

# Before
3. Next, let's try on a slightly harder example. Below is a simple script that trains a support vector machine on the
    iris dataset.

# After
3. Next, let's try on a slightly harder example. Below is a simple script that trains a support vector machine on the
    breast cancer dataset.

Visual Change:
Documentation Change

The exercise files remain unchanged and continue to use load_breast_cancer() from sklearn.datasets.

Original prompt

This section details on the original issue you should resolve

<issue_title>[bug] CLI exercise classifier dataset mismatch</issue_title>
<issue_description>In exercise of Command line arguments at s2_organisation_and_version_control/cli/, part 3 mentions:

... trains a support vector machine on the iris dataset.

However the loaded dataset in the train script is load_breast_cancer() from sklearn.datasets</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Fix dataset mismatch in CLI exercise classification Fix dataset mismatch in CLI exercise files Jan 12, 2026
Copilot AI requested a review from SkafteNicki January 12, 2026 07:26
@SkafteNicki
Copy link
Owner

@copilot instead of changing the exercise files please just change the learning module to state that the iris dataset is used

Copy link
Contributor Author

Copilot AI commented Jan 16, 2026

@copilot instead of changing the exercise files please just change the learning module to state that the iris dataset is used

Done! Updated the documentation instead. The exercise files now remain unchanged (using load_breast_cancer()), and the learning module in cli.md line 195 has been updated to say "breast cancer dataset" instead of "iris dataset".

Commit: f810833

Documentation Change

Copilot AI changed the title Fix dataset mismatch in CLI exercise files Fix dataset mismatch in CLI exercise documentation Jan 16, 2026
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.

[bug] CLI exercise classifier dataset mismatch

2 participants