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

Mesa 2.1.1 updates to tutorials for CSSSA tutorial and history update #1744

Merged
merged 2 commits into from
Jul 30, 2023
Merged

Mesa 2.1.1 updates to tutorials for CSSSA tutorial and history update #1744

merged 2 commits into from
Jul 30, 2023

Conversation

tpike3
Copy link
Member

@tpike3 tpike3 commented Jul 29, 2023

  • update README screenshot
  • add Colab badge to intro tutorial
  • update visualization tutorial so it references git install of mesa-models

@codecov
Copy link

codecov bot commented Jul 29, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.05% 🎉

Comparison is base (f1e3e97) 72.29% compared to head (b7bce4e) 72.35%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1744      +/-   ##
==========================================
+ Coverage   72.29%   72.35%   +0.05%     
==========================================
  Files          20       20              
  Lines        1303     1302       -1     
  Branches      246      246              
==========================================
  Hits          942      942              
+ Misses        326      325       -1     
  Partials       35       35              
Files Changed Coverage Δ
mesa/__init__.py 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rht
Copy link
Contributor

rht commented Jul 29, 2023

I'm not a Tufte expert, but the new one has a high "data-to-ink" ratio. But:

  1. it doesn't show the current selected number of agents.
  2. it doesn't showcase a Jupyter notebook analysis.

For point 2, not sure it is necessary, given that the visualization tutorial already showcase it. But this is meant to be a screenshot that shows people as much context as possible.

@tpike3
Copy link
Member Author

tpike3 commented Jul 29, 2023

I'm not a Tufte expert, but the new one has a high "data-to-ink" ratio. But:

  1. it doesn't show the current selected number of agents.
  2. it doesn't showcase a Jupyter notebook analysis.

For point 2, not sure it is necessary, given that the visualization tutorial already showcase it. But this is meant to be a screenshot that shows people as much context as possible.

I went back and forth on how to update it, in the end the I went with just a screenshot of the visualization as it has substantially less information to process, but that did make me realized I need to update the caption and I can put the jupyter notebook context in there. I will also update so it included the number of agents

- add Colab badge to intro tutorial and make some minor updates
- update visualization tutorial so it references git install of mesa-models
@tpike3 tpike3 added this to the Mesa 2.1.1 (Avondale) milestone Jul 30, 2023
HISTORY.rst Outdated Show resolved Hide resolved
@tpike3 tpike3 changed the title Minor fixes and updates for CSSSA tutorial Mesa 2.1.1 updates to tutorials for CSSSA tutorial and history update Jul 30, 2023
@rht rht merged commit 2cec830 into projectmesa:main Jul 30, 2023
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.

3 participants