desihub / desisurvey

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

allow programs other than DARK, GRAY, BRIGHT #84

Closed sbailey closed 5 years ago

sbailey commented 6 years ago

There are several places throughout desisurvey that hardcode the known programs to be 'DARK', 'GRAY', or 'BRIGHT'. We may need more flexibility for commissioning and SV. Consider generalizing this so that the set of programs used is derived from the programs listed in the config.yaml file (which should also be a superset of the programs in the input tiles file).

This could get tricky, so if it starts taking a lot of time or complexity, re-evaluate whether this flexibility is really needed.

Related to PR #83 .

dkirkby commented 6 years ago

Thinking about this a bit more, the scheduler associates DARK/GRAY/BRIGHT with corresponding cuts on the sun & moon parameters, so these program names are effectively predefined in the code and used to ensure that a DARK tile is not observed when the moon is up, etc.

I can think of two possible SV scenarios that would require new program definitions:

Does either of these apply to SV? Otherwise, can someone describe the SV use case where you wouldn't want to label tiles using DARK/GRAY/BRIGHT?

dkirkby commented 5 years ago

Closing this with the "wontfix" label since the current desisurvey.tiles.Tiles assumptions should be sufficiently general to support SV. Please reopen with any new info.