search

nkremerh / sugarscape

Development repository for the Digital Terraria Lab implementation of the Sugarscape agent-based societal simulation.
https://github.com/digital-terraria-lab/sugarscape
MIT License
7 stars 12 forks source link

Removes duplicated addition of universal income every timestep #27

Closed colinhanrahan closed 8 months ago

colinhanrahan commented 8 months ago

self.sugar += self.universalSugar and self.spice += self.universalSpice are already present in doUniversalIncome() which is called in doTimestep(). Based on the time interval logic in doUniversalIncome(), it doesn’t seem like universalSugar and universalSpice should be added every single timestep in addition to the call to doUniversalIncome().

nkremerh commented 8 months ago

@mmilkowski36 Please review.