Skip to content

Conversation

@unnati-jaiswal24
Copy link

Thanks for opening a PR! Please click the Preview tab and select a PR template:

  • [🐛 Bug fix]
  • [🛠 Feature/enhancement]
  • fixed the installation issues for zsh and its universal for all terminal, command prompt ,etc.
  • fixed the suggestions.

@EwoutH
Copy link
Member

EwoutH commented Nov 12, 2025

Thanks for the PR. Can you explain what it does and why it does it following our PR template?

@github-actions
Copy link

Performance benchmarks:

Model Size Init time [95% CI] Run time [95% CI]
BoltzmannWealth small 🔵 +0.3% [-1.1%, +1.6%] 🔵 -0.6% [-0.7%, -0.4%]
BoltzmannWealth large 🔵 -0.9% [-2.5%, +0.2%] 🟢 -6.5% [-9.1%, -4.0%]
Schelling small 🔵 -0.5% [-0.7%, -0.3%] 🔵 -0.7% [-0.9%, -0.6%]
Schelling large 🔵 -1.6% [-2.2%, -1.1%] 🔵 -2.5% [-4.5%, -0.5%]
WolfSheep small 🔵 -1.6% [-2.1%, -1.2%] 🔵 +0.1% [-0.1%, +0.3%]
WolfSheep large 🔵 +0.6% [-0.1%, +1.4%] 🔵 +1.9% [+0.4%, +3.3%]
BoidFlockers small 🔵 +3.0% [+2.3%, +3.7%] 🔵 +0.1% [-0.1%, +0.3%]
BoidFlockers large 🔵 +3.2% [+2.5%, +3.7%] 🔵 -0.6% [-1.0%, -0.3%]

@unnati-jaiswal24
Copy link
Author

unnati-jaiswal24 commented Nov 12, 2025

@EwoutH here's given description of what i fixed.

On macOS and other systems💻 using zsh, the command pip install mesa[viz] causes the error😵
zsh: no matches found: mesa[viz] because zsh interprets square brackets[ ] as pattern matching.

using zsh, the unquoted command causes the error:
zsh: no matches found: mesa[viz]

Quoting the package 📦name fixes the issue and makes installation smoother for new users.
❌removed: mesa[viz] / mesa[rec]
✅added: "mesa[viz]"/ "mesa[rec]"

Helps new user to run in their zsh and also this change is universal🌎 and safe on all operating systems. It prevent shell related errors ✅.

@EwoutH
Copy link
Member

EwoutH commented Nov 13, 2025

Thanks!

Could you resolve the git conflicts with the main branch (preferably by rebasing)?

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.

2 participants