Skip to content

Revert "Deprecate model.steps in favor of model.time"#3151

Merged
EwoutH merged 1 commit intomesa:mainfrom
EwoutH:revert_depr_steps
Jan 16, 2026
Merged

Revert "Deprecate model.steps in favor of model.time"#3151
EwoutH merged 1 commit intomesa:mainfrom
EwoutH:revert_depr_steps

Conversation

@EwoutH
Copy link
Copy Markdown
Member

@EwoutH EwoutH commented Jan 16, 2026

This reverts commit 7a9299c and PR (#3136).

I see a role for steps after all, and I don't think it should be removed.

Took some black magic, will show in a bit.

@EwoutH EwoutH added the ignore-for-release PRs that aren't included in the release notes label Jan 16, 2026
@EwoutH EwoutH merged commit bf76c1c into mesa:main Jan 16, 2026
14 of 15 checks passed
@github-actions
Copy link
Copy Markdown

Performance benchmarks:

Model Size Init time [95% CI] Run time [95% CI]
BoltzmannWealth small 🔵 +0.3% [-0.1%, +0.8%] 🔵 -0.1% [-0.3%, -0.0%]
BoltzmannWealth large 🔵 -0.1% [-1.4%, +0.9%] 🔵 +1.9% [+0.7%, +3.5%]
Schelling small 🔵 +0.2% [+0.0%, +0.4%] 🔵 -0.8% [-0.9%, -0.7%]
Schelling large 🔵 +1.3% [+0.7%, +2.0%] 🔵 -0.8% [-2.7%, +1.4%]
WolfSheep small 🔵 +0.2% [-0.1%, +0.5%] 🔵 -1.1% [-1.4%, -0.9%]
WolfSheep large 🔵 -1.3% [-2.0%, -0.8%] 🔵 -3.3% [-4.6%, -2.0%]
BoidFlockers small 🔵 +0.9% [+0.3%, +1.5%] 🔵 -0.1% [-0.3%, +0.1%]
BoidFlockers large 🔵 +1.8% [+1.3%, +2.5%] 🔵 -0.3% [-0.6%, -0.1%]

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

Labels

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.

1 participant