Skip to content

docs: fix typo in first model tutorial#3249

Merged
EwoutH merged 1 commit intomesa:mainfrom
ApurvaPatil2401:fix-tutorial-typo
Feb 6, 2026
Merged

docs: fix typo in first model tutorial#3249
EwoutH merged 1 commit intomesa:mainfrom
ApurvaPatil2401:fix-tutorial-typo

Conversation

@ApurvaPatil2401
Copy link
Copy Markdown
Contributor

This PR fixes a minor typographical error in the 0_first_model.ipynb tutorial.

Changed: "datacollection" → "data collection"

I have verified my development environment by running the SolarViz tests (pytest tests/visualization/test_solara_viz.py), which passed successfully.

This is my first contribution to Mesa as part of my GSoC 2026 preparation!

@codebreaker32
Copy link
Copy Markdown
Collaborator

Maybe that datacollection referred to the mesa/datacollection.py

@ApurvaPatil2401
Copy link
Copy Markdown
Contributor Author

Hi @codebreaker32, thank you for the review! I considered that, but since 'datacollection' is listed alongside 'creation' and 'activation' as general tasks the manager class performs, I thought 'data collection' (two words) read more naturally as a description of the action. If the intent was to refer specifically to the module, should we perhaps change it to datacollection.py or format it as code so it's clear to the reader?

Copy link
Copy Markdown
Member

@EwoutH EwoutH left a comment

Choose a reason for hiding this comment

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

I think it's indeed clearer this way, thanks!

@EwoutH EwoutH added docs Release notes label ignore-for-release PRs that aren't included in the release notes labels Feb 6, 2026
@EwoutH EwoutH merged commit 73f93e6 into mesa:main Feb 6, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Release notes label ignore-for-release PRs that aren't included in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants