Skip to content

Small draw_space docstring fix #2554

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jan 1, 2025
Merged

Small draw_space docstring fix #2554

merged 1 commit into from
Jan 1, 2025

Conversation

quaquel
Copy link
Member

@quaquel quaquel commented Dec 19, 2024

The docstring for draw_space contained the description of a keyword argument that was not used. This removes that.

@quaquel quaquel added the docs Release notes label label Dec 19, 2024
Copy link

Performance benchmarks:

Model Size Init time [95% CI] Run time [95% CI]
BoltzmannWealth small 🔵 -0.5% [-1.2%, +0.2%] 🔵 -0.7% [-1.1%, -0.3%]
BoltzmannWealth large 🔵 -0.2% [-0.7%, +0.3%] 🔵 -0.6% [-0.8%, -0.3%]
Schelling small 🔵 -0.5% [-0.8%, -0.3%] 🔵 -0.6% [-0.7%, -0.5%]
Schelling large 🔵 -3.9% [-12.2%, +0.4%] 🔵 -0.0% [-0.9%, +0.9%]
WolfSheep small 🔵 +0.5% [+0.2%, +0.7%] 🔵 +0.5% [-0.6%, +1.6%]
WolfSheep large 🔵 +0.6% [+0.1%, +1.0%] 🔵 +0.9% [-0.2%, +2.0%]
BoidFlockers small 🔵 +0.9% [+0.3%, +1.6%] 🔵 +1.1% [+0.4%, +2.0%]
BoidFlockers large 🔵 +1.0% [+0.5%, +1.5%] 🔵 +1.6% [+1.1%, +2.1%]

@tpike3 tpike3 merged commit 22a1807 into projectmesa:main Jan 1, 2025
13 checks passed
@EwoutH
Copy link
Member

EwoutH commented Jan 1, 2025

Sorry totally missed this one, thanks for reviewing and merging.

@quaquel quaquel deleted the doc_bug branch February 12, 2025 18:46
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
docs Release notes label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants