Skip to content

Allow list inputs for MesaSignal observable names and signal types#3139

Merged
quaquel merged 8 commits intomesa:mainfrom
Sonu0305:fixme_mesa_signal
Jan 17, 2026
Merged

Allow list inputs for MesaSignal observable names and signal types#3139
quaquel merged 8 commits intomesa:mainfrom
Sonu0305:fixme_mesa_signal

Conversation

@Sonu0305
Copy link
Copy Markdown
Contributor

Summary

Updated MesaSignal methods (observe, unobserve, clear_all_subscriptions) in the experimental signals module to accept a list of strings for name and signal_type arguments, in addition to single strings or All.

Motive

This enhancement resolves a FIXME in mesa_signal.py.

Implementation

Usage Examples

Additional Notes

@github-actions
Copy link
Copy Markdown

Performance benchmarks:

Model Size Init time [95% CI] Run time [95% CI]
BoltzmannWealth small 🔵 -0.7% [-1.8%, +0.5%] 🔵 +1.2% [+1.1%, +1.4%]
BoltzmannWealth large 🔵 -0.0% [-0.8%, +0.9%] 🔵 -3.6% [-5.2%, -2.0%]
Schelling small 🔵 +0.5% [+0.2%, +0.8%] 🔵 +1.5% [+1.4%, +1.7%]
Schelling large 🔵 +0.9% [+0.3%, +1.5%] 🔵 -0.9% [-2.3%, +0.4%]
WolfSheep small 🔵 -0.9% [-1.2%, -0.5%] 🔵 +0.9% [+0.6%, +1.3%]
WolfSheep large 🔵 -1.4% [-2.3%, -0.3%] 🔵 -2.7% [-4.4%, -1.2%]
BoidFlockers small 🔵 +1.8% [+1.3%, +2.2%] 🔵 +2.1% [+1.9%, +2.2%]
BoidFlockers large 🔵 +2.1% [+1.5%, +2.6%] 🔵 +1.2% [+0.9%, +1.5%]

@Sonu0305 Sonu0305 marked this pull request as draft January 16, 2026 17:01
@Sonu0305 Sonu0305 marked this pull request as ready for review January 16, 2026 17:06
@Sonu0305 Sonu0305 requested a review from quaquel January 16, 2026 17:06
@github-actions
Copy link
Copy Markdown

Performance benchmarks:

Model Size Init time [95% CI] Run time [95% CI]
BoltzmannWealth small 🔵 -2.3% [-3.1%, -1.5%] 🔵 -0.5% [-0.7%, -0.4%]
BoltzmannWealth large 🔵 +0.5% [-1.0%, +1.7%] 🔵 +0.3% [-1.2%, +1.5%]
Schelling small 🔵 +0.9% [+0.7%, +1.1%] 🔵 -0.2% [-0.3%, -0.1%]
Schelling large 🔵 -0.6% [-0.9%, -0.2%] 🟢 -3.9% [-4.7%, -3.0%]
WolfSheep small 🔵 -0.3% [-0.6%, -0.0%] 🔵 -1.1% [-1.3%, -1.0%]
WolfSheep large 🔵 +2.8% [+1.1%, +4.4%] 🔵 +2.1% [-0.4%, +4.5%]
BoidFlockers small 🔵 +2.9% [+2.1%, +3.6%] 🔵 -0.4% [-0.5%, -0.2%]
BoidFlockers large 🔴 +4.0% [+3.5%, +4.5%] 🔵 -0.4% [-0.8%, -0.1%]

@quaquel quaquel added the enhancement Release notes label label Jan 16, 2026
@quaquel
Copy link
Copy Markdown
Member

quaquel commented Jan 17, 2026

I merged another PR on signals, so can you resolve the conflicts? I think I can merge this quickly afterwards.

@Sonu0305
Copy link
Copy Markdown
Contributor Author

I merged another PR on signals, so can you resolve the conflicts? I think I can merge this quickly afterwards.

Sure, on it.

@quaquel quaquel merged commit ba3abe8 into mesa:main Jan 17, 2026
14 checks passed
@Sonu0305 Sonu0305 deleted the fixme_mesa_signal branch January 17, 2026 15:26
Sonu0305 added a commit to Sonu0305/mesa that referenced this pull request Jan 17, 2026
quaquel pushed a commit that referenced this pull request Jan 17, 2026
quaquel pushed a commit to quaquel/mesa that referenced this pull request Jan 18, 2026
quaquel pushed a commit to quaquel/mesa that referenced this pull request Jan 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Release notes label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants