Renderer null check fix in solaraviz by Sahil-Chhoker · Pull Request #2809 · mesa/mesa

Sahil-Chhoker · 2025-07-10T05:46:41Z

Explicity check if the renderer is None or not in SolaraViz.
Normal check was causing a error on the solara interface.

github-actions · 2025-07-10T05:55:04Z

Performance benchmarks:

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

tpike3

LGTM

renderer null check fix in solaraviz

de55a80

Sahil-Chhoker requested a review from tpike3 July 10, 2025 05:47

tpike3 approved these changes Jul 11, 2025

View reviewed changes

tpike3 merged commit f9e6dbb into mesa:main Jul 11, 2025
13 checks passed

tpike3 added visualisation enhancement Release notes label labels Aug 30, 2025

Provide feedback