Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-27863

Allow controller = 'H' for exposure_id calculation

    XMLWordPrintable

Details

    • Improvement
    • Status: Done
    • Resolution: Done
    • None
    • obs_lsst

    Description

      To allow ingest of phosim-simulated data while using lsstCam mapper we suggest to add  controller = 'H'.  This is to guarantee unique obsid values for phosim data (Tim Jennes's comment).  If this is done, currently the translator returns (using  obs_lsst version w.2020.48-1-gf0a7a3e )

      translate_header.py -p lsst.obs.lsst.translators MC_H_20000217_000032_R00_SW0.fits 
       
      Analyzing MC_H_20000217_000032_R00_SW0.fits...
      ValueError("Supplied controller, 'H' is neither 'O' nor 'C'")
      

      An update is needed to allow for controller = 'H' . It affects the  `exposure_id` method in https://github.com/lsst/obs_lsst/blob/master/python/lsst/obs/lsst/translators/lsst.py  

      It doesn't clash with other definitions https://confluence.lsstcorp.org/display/SYSENG/Image+naming+and+visit+definition+conventions (K-T Lim's comment). 

       

      Attachments

        Issue Links

          Activity

            People

              ksuberlak Krzysztof Suberlak
              ksuberlak Krzysztof Suberlak
              Kian-Tat Lim
              Kian-Tat Lim, Krzysztof Suberlak, Robert Lupton, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.