search

projectmesa / mesa-examples

Seminal agent-based models developed using Mesa
Other
116 stars 140 forks source link

fix: sugarscape G1MT: Remove redundant schedule steps increment #92

Closed rht closed 8 months ago

rht commented 8 months ago

cc: @tpike3

EwoutH commented 8 months ago

Could you add a brief description what changed exactly that this isn't needed anymore? That helps to place it in context.

rht commented 8 months ago

This is a long-standing bug. The RandomActivationByType already has https://github.com/projectmesa/mesa/blob/c0c8479e4d1f29c20ee2a32caeb6a3eaa1682041/mesa/time.py#L363.

rht commented 8 months ago

Wait, never mind. self.schedule.steps() never gets called. This is probably not the right fix. @tpike3 may know the problem better.