mesa icon indicating copy to clipboard operation
mesa copied to clipboard

Dedicated widget for seed control

Open quaquel opened this issue 1 year ago • 8 comments

As discussed in #2496, having a clean dedicated widget for seed control would be nice. This should allow for controlling the seed and falling back on None (so fully random) if desired.

quaquel avatar Nov 13 '24 19:11 quaquel

can I take this issue?

nissu99 avatar Dec 09 '24 19:12 nissu99

@nissu99 Yes. Two things to watch out for:

  • https://github.com/projectmesa/mesa/pull/2540 is touching the same part of the codebase, which could be cause a merge conflict if you also open a PR. You can either watch till that one is merged, or resolve the conflict when it happens.
  • We need to agree on an API / user interface. My proposal is that this seed widget is enabled by default for all SolaraViz models, but I think there were some different ideas on this.

EwoutH avatar Dec 09 '24 19:12 EwoutH

sorry for acting late on this issue ,my exams just got over.

nissu99 avatar Jan 09 '25 14:01 nissu99

@EwoutH do we also need a component to track the seed state,read discussion from @quaquel in #2496 regarding this.

nissu99 avatar Jan 09 '25 14:01 nissu99

No worries, hope they went well!

I don't think seed state tracking is required at this moment, but I will also let @quaquel pitch in.

EwoutH avatar Jan 09 '25 16:01 EwoutH

I would like to have it back at some point, but there is no rush.

quaquel avatar Jan 10 '25 08:01 quaquel

how does this look --

seed when set random

Image

seed changed manually

Image

@quaquel @EwoutH @Corvince how is this checkbox type design?

nissu99 avatar Jan 17 '25 05:01 nissu99

Design wise I think this is nice, @quaquel any comments? Do we want/need to support seeds other than integers?

EwoutH avatar Jan 23 '25 19:01 EwoutH