Skip to content

Remove model.steps usage from solara_viz#3344

Merged
quaquel merged 1 commit intomesa:mainfrom
codebreaker32:main
Feb 18, 2026
Merged

Remove model.steps usage from solara_viz#3344
quaquel merged 1 commit intomesa:mainfrom
codebreaker32:main

Conversation

@codebreaker32
Copy link
Copy Markdown
Collaborator

@codebreaker32 codebreaker32 commented Feb 18, 2026 •
edited
Loading

Remove the steps usage from solara_viz likely left from #3328

Error encountered:

Traceback (most recent call last):
  File "/home/aman/mesa/env/lib/python3.12/site-packages/reacton/core.py", line 1702, in _render
    root_element = el.component.f(*el.args, **el.kwargs)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/aman/mesa/mesa/visualization/solara_viz.py", line 1038, in ShowSteps
    return solara.Text(f"Time: {model.steps}")
                                ^^^^^^^^^^^
AttributeError: 'VirusOnNetwork' object has no attribute 'steps'. Did you mean: 'step'?

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 18, 2026

Performance benchmarks:

Model Size Init time [95% CI] Run time [95% CI]
BoltzmannWealth small 🔵 -1.3% [-1.7%, -0.9%] 🔵 -2.0% [-2.4%, -1.6%]
BoltzmannWealth large 🔵 -1.3% [-1.6%, -1.1%] 🔵 -2.8% [-3.0%, -2.4%]
Schelling small 🔵 -0.2% [-0.3%, -0.1%] 🔵 -0.9% [-1.1%, -0.8%]
Schelling large 🔵 +0.5% [+0.2%, +0.7%] 🔵 -0.1% [-0.3%, +0.1%]
WolfSheep small 🔵 -0.4% [-0.6%, -0.2%] 🔵 -0.6% [-0.8%, -0.4%]
WolfSheep large 🔵 -0.7% [-1.4%, -0.1%] 🔵 +0.1% [-0.2%, +0.3%]
BoidFlockers small 🔵 +1.8% [+1.4%, +2.3%] 🔵 -0.5% [-0.7%, -0.3%]
BoidFlockers large 🔵 +1.6% [+0.9%, +2.2%] 🔵 -1.1% [-1.8%, -0.4%]

@codebreaker32 codebreaker32 changed the title Initial Commit Remove steps from solara_viz Feb 18, 2026
@codebreaker32 codebreaker32 changed the title Remove steps from solara_viz Remove model.steps usage from solara_viz Feb 18, 2026
@quaquel quaquel added bug Release notes label backport-candidate PRs we might want to backport to an earlier branch labels Feb 18, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 18, 2026

Caution

Review failed

An error occurred during the review process. Please try again later.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch main

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@quaquel quaquel removed the backport-candidate PRs we might want to backport to an earlier branch label Feb 18, 2026
@quaquel quaquel merged commit f1a4693 into mesa:main Feb 18, 2026
16 of 17 checks passed
Krishsharma179 pushed a commit to Krishsharma179/mesa that referenced this pull request Feb 21, 2026
@codebreaker32 @Krishsharma179
Initial Commit (mesa#3344)
dc85e39
EwoutH pushed a commit that referenced this pull request Mar 15, 2026
@codebreaker32 @EwoutH
Remove model.steps usage from solara_viz (#3344)
0f56ac8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@quaquel quaquel quaquel approved these changes

Assignees

No one assigned

Labels

bug Release notes label

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@codebreaker32 @quaquel