desihub/desidatamodel

Rename SURVEYOPS to DESI_SURVEYOPS

Closed this issue · 1 comments

weaverba137 commented

The data model directory SURVEYOPS actually points to the same place as DESI_SURVEYOPS. In order to avoid confusion and in keeping with our convention of using environment variables as top-level directories, I propose we rename the data model directory to DESI_SURVEYOPS.

weaverba137 commented

Based on discussions with @sbailey on Slack, here is the full proposal:

  1. Rename SURVEYOPS -> DESI_SURVEYOPS in the data model. This environment variable is actually used, and neatly folds the version (trunk, tags/1.0, etc) into the name.
  2. Create DESI_SURVEY analogous to DESI_TARGET.
  3. Put requested intermediate fiberassign files (#103) under DESI_SURVEY/fiberassign.
  4. Add DESI_SURVEY to desitree.
  5. DESISURVEY_OUTPUT is the output of desisurvey and surveysim, but it doesn't necessarily live under DESI_SURVEY. Consider opening a separate ticket for this. Additional data point: loading the desisurvey and surveysim modules does not define DESISURVEY_OUTPUT.