Skip to content

Remove progress bar feature #834

@RobertJacobsonCDC

Description

@RobertJacobsonCDC

The progress bar seemed like a good idea that just didn't go anywhere:

  • Progress to max_time isn't especially useful, because usually max_time is some arbitrarily large value that is chosen to be large enough to capture the time window of any reasonable simulation run. In other words, it's not related to the actual simulation run time.
  • Even for many simple models, there is no natural custom definition of progress. You might take percentage of total population that has been infected, for example. But: "even for highly infectious pathogens in a perfectly mixed population, the theoretical epidemic size is less than 100% of the population, and this is much more true for less infectious pathogens and for more structured host populations." (See this.)

So there doesn't seem to be a useful default notion of progress, and even for simple models there doesn't seem to be any reasonable alternative. Add to this the fact that our current implementation redraws unconditionally, causing a severe performance regression when enabled—the case for removing it is pretty strong.

Metadata

Metadata

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions