mesa icon indicating copy to clipboard operation
mesa copied to clipboard

SolaraViz: Remove card title

Open rht opened this issue 1 year ago • 5 comments

They are soon going to be part of one umbrella of "measures", instead of distinctly named ones. If a user wants to add a title, they can do it inside Matplotlib/Altair.

rht avatar Aug 23 '24 10:08 rht

Performance benchmarks:

Model Size Init time [95% CI] Run time [95% CI]
Schelling small 🔵 -0.1% [-0.4%, +0.2%] 🔵 -0.0% [-0.2%, +0.2%]
Schelling large 🔵 +0.2% [-0.4%, +0.7%] 🔵 -1.0% [-1.6%, -0.4%]
WolfSheep small 🔵 -0.5% [-1.7%, +0.7%] 🔵 -1.3% [-1.6%, -1.0%]
WolfSheep large 🔵 -0.4% [-0.7%, -0.0%] 🔵 -0.8% [-1.1%, -0.6%]
BoidFlockers small 🔵 +0.8% [+0.1%, +1.7%] 🔵 -0.2% [-1.1%, +0.7%]
BoidFlockers large 🔵 -0.3% [-1.1%, +0.6%] 🔵 -0.0% [-0.4%, +0.4%]

github-actions[bot] avatar Aug 23 '24 11:08 github-actions[bot]

@rht Thanks! -- Could you elaborate on "They are soon going to be part of one umbrella of measures". Is this going to be a change to Mesa, Solara or both?

Could you add to the visualization tutorial adding names to the plots? I think this is a small but critical feature for users.

tpike3 avatar Aug 26 '24 09:08 tpike3

How can I help move this forward?

EwoutH avatar Aug 30 '24 15:08 EwoutH

Could you elaborate on "They are soon going to be part of one umbrella of measures"

https://github.com/projectmesa/mesa/discussions/2236, and in https://github.com/projectmesa/mesa/pull/2263, where measures and space are merged into "components" (what I earlier called measures).

Could you add to the visualization tutorial adding names to the plots? I think this is a small but critical feature for users.

Currently there is no public API for that, and this PR doesn't add/remove any feature in this area.

How can I help move this forward?

This PR just needs a clarification answer to @tpike3 's questions. Unless there is other missing aspect.

rht avatar Aug 31 '24 17:08 rht

@rht @Corvince what's the current state / trajectory of this PR?

EwoutH avatar Sep 20 '24 08:09 EwoutH

This PR is outdated and in the revised solara_viz this code is not used anymore.

quaquel avatar Oct 23 '24 19:10 quaquel