Skip to content

fix: Apply default value to slider #2016

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 29, 2024
Merged

fix: Apply default value to slider #2016

merged 1 commit into from
Jan 29, 2024

Conversation

rht
Copy link
Contributor

@rht rht commented Jan 29, 2024

Without this, the slider value always defaults to being leftmost.

Copy link

Performance benchmarks:

Model Size Init time [95% CI] Run time [95% CI]
Schelling small 🔵 +0.0% [-0.3%, +0.4%] 🔵 +0.1% [-0.2%, +0.4%]
Schelling large 🔵 +0.1% [-1.2%, +1.1%] 🔵 -1.0% [-2.4%, +0.7%]
WolfSheep small 🔵 -0.0% [-0.3%, +0.3%] 🔵 -0.1% [-0.2%, +0.0%]
WolfSheep large 🔵 +0.7% [-0.1%, +1.5%] 🔵 +1.2% [+0.1%, +2.5%]
BoidFlockers small 🔵 -0.4% [-0.8%, -0.0%] 🔵 -0.7% [-1.2%, -0.1%]
BoidFlockers large 🔵 -0.2% [-0.5%, +0.0%] 🔵 -0.5% [-1.0%, -0.0%]

@rht rht added the bug Release notes label label Jan 29, 2024
@rht rht merged commit 2837cc1 into projectmesa:main Jan 29, 2024
@rht rht deleted the fix_slider branch January 29, 2024 08:05
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Release notes label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants