Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Added aliases for terms in viz modules #2634

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

sanika-n
Copy link
Contributor

In one of the comments present in the code review for this PR #2430 , there was a request to make some of the aliases related to the visualization module centralized, this PR aims to do that. The link can be found here
image
@EwoutH, I am not exactly sure if you meant to make it centralized throughout mesa or just in the visualization module. Since all the aliases were related to the visualization module, I have added it there.

Copy link

Performance benchmarks:

Model Size Init time [95% CI] Run time [95% CI]
BoltzmannWealth small 🟢 -11.5% [-12.8%, -10.0%] 🔵 +1.1% [+0.9%, +1.2%]
BoltzmannWealth large 🔵 +0.5% [-1.2%, +3.1%] 🔵 -0.7% [-3.3%, +2.0%]
Schelling small 🔵 -0.8% [-1.2%, -0.3%] 🔵 -0.2% [-0.4%, -0.1%]
Schelling large 🔵 -0.6% [-1.1%, -0.2%] 🔵 -0.6% [-2.0%, +1.0%]
WolfSheep small 🔵 -2.3% [-2.7%, -1.9%] 🔵 +0.5% [+0.3%, +0.7%]
WolfSheep large 🔵 -0.9% [-1.7%, -0.1%] 🔵 -2.6% [-4.9%, -0.4%]
BoidFlockers small 🔵 -0.7% [-1.3%, -0.1%] 🔵 +0.5% [+0.2%, +0.7%]
BoidFlockers large 🔵 -0.9% [-1.5%, -0.3%] 🔵 +0.6% [+0.3%, +0.8%]

@quaquel
Copy link
Member

quaquel commented Jan 26, 2025

I am not sure about the need for this PR, although I realize this was suggested by @EwoutH.

Why might this not be needed

  1. old style spaces are on there way out, simplifying the typing needed
  2. Ideally, whether one uses matplotlib or altair (or plotly perhaps in the future) becomes a back-end choice. There should thus be only a single entry point for drawing spaces.

@EwoutH
Copy link
Member

EwoutH commented Jan 26, 2025

It's always good advice to check with (multiple) maintainers if something is necessary / useful / worthwhile before working on something. This can be a simple as a quick message in the Matrix chat or a follow-up comment on a PR or issue.

I don't have a strong opinion on this I will leave this to be @quaquel's call.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants