Skip to content

[Docs] Fix typos and inconsistent comments in tutorials#2948

Merged
tpike3 merged 2 commits intomesa:mainfrom
Mannan-15:docs/fix-typos-tutorials
Dec 15, 2025
Merged

[Docs] Fix typos and inconsistent comments in tutorials#2948
tpike3 merged 2 commits intomesa:mainfrom
Mannan-15:docs/fix-typos-tutorials

Conversation

@Mannan-15
Copy link
Copy Markdown
Contributor

@Mannan-15 Mannan-15 commented Dec 11, 2025

Summary

I noticed a few typos and inconsistencies while working through the introductory tutorials. This PR fixes them to improve clarity for new users.

Changes

  1. docs/tutorials/1_adding_space.ipynb:

    • Fixed a discrepancy where the text description stated cell capacity was 5, but the code initialized it as 10. Updated the code to match the text (5).
  2. docs/tutorials/2_collecting_data.ipynb:

    • Fixed typos in the comments: welath -> wealth and agentat -> agent at.

Context

Hi! I'm an undergrad from IIT Dhanbad interested in contributing to Mesa for GSoC 2026. This is my first PR to familiarize myself with the workflow & I have some ideas to propose on Behavioral framework.

@EwoutH EwoutH requested a review from tpike3 December 11, 2025 10:51
@EwoutH EwoutH added the docs Release notes label label Dec 11, 2025
@tpike3
Copy link
Copy Markdown
Member

tpike3 commented Dec 14, 2025

@Mannan-15 Thanks for the PR! The changes look fine, but I think the build is failing based on our recent repo name change. Let me look into that real quick so we can ensure your PRs passes its tests.

@Mannan-15
Copy link
Copy Markdown
Contributor Author

Thanks for the review @tpike3!

Glad to hear the changes look good. I'll wait for the CI/CD build issues to be resolved.
In the meantime, are there any other specific issues (in mesa or mesa_llm) that you would recommend? I'm eager to keep contributing while this gets merged.

@tpike3
Copy link
Copy Markdown
Member

tpike3 commented Dec 15, 2025

@Mannan-15 As the build failures do not involve your code i am going to merge this. As for things to do I would engage in the mesa-llm repo and with that team. As llm's are very popular mesa-llm has a lot of people engaged so I would definitely work with them to find a spot they need contributions.

One thing mesa really needs is its examples repo updated so they reflect 3.3. An AI area to start would be the reinforcement learning integration if you want to learn about multi agent reinforcement learning. If you do examples please ping us in the chat https://github.com/projectmesa so we know you did it.

@tpike3 tpike3 merged commit 310e304 into mesa:main Dec 15, 2025
11 of 25 checks passed
@Mannan-15
Copy link
Copy Markdown
Contributor Author

Thanks for the merge @tpike3!
I’ll definitely engage with the mesa-llm as you suggested.
Regarding mesa-examples, I'm very interested in the multi-agent reinforcement learning integration. I plan to work on developing new examples that utilize Mesa 3.3 capabilities and start prototyping the RL integration features.

I'll make sure to ping you in the chat once I have some progress to share. (but the link you provided seems to redirect to the wrong page).

@EwoutH
Copy link
Copy Markdown
Member

EwoutH commented Dec 17, 2025

This PR broke our CI and the tutorial for most users:

As the build failures do not involve your code i am going to merge this.

While for the build failures this was true, it unfortunately wasn't for the Readthedocs failure.

But it helped discover an imperfection in both our cell movement handling, so hey.

@Mannan-15
Copy link
Copy Markdown
Contributor Author

Ah, thanks for catching that @EwoutH! Sorry about the tutorial breakage, but I'm glad it at least helped surface the edge case in the movement logic.

Thanks for handling the fix. I'll keep an eye on the discussion in #2962 to see how the capacity handling evolves.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Release notes label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants