desihub / desisurvey

Code for desi survey planning and implementation
BSD 3-Clause "New" or "Revised" License
2 stars 7 forks source link

Remove simulate blocks from desisurvey #119

Open schlafly opened 4 years ago

schlafly commented 4 years ago

As part of making desisurvey.scheduler usable on the mountain, desisurvey.plan picked up some "simulate" blocks that are only used if surveysim calls the planner with the simulate argument.

It's unfortunate to have any "simulation" code in desisurvey proper. We could imagine moving much of the simulated fiberassignment scheme out of desisurvey and into surveysim, to keep desisurvey free of simulation items. In this case, surveysim would need to keep track of things like COUNTDOWN, which is used in simulating fiber assignment, instead of desisurvey.