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

add the REB_COND and CONFIG_COND FITS headers to metadata if they are present in the file

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: obs_lsst
    • Labels:
      None
    • Team:
      External

      Description

      at present, some important state about the CCD conditions is captured in a FITS header called REB_COND in auxtel data. In addition, there are checksums about sequencer files etc captured in a header called CONFIG_COND. Neither currently appear to be available from , for example a call to 

      lsst.afw.image.exposure.ExposureF.info.getMetadata()
      

      I think though of course am not certain that this can be added by overriding the readMetadata() method for the class LatissRawFormatter in https://github.com/lsst/obs_lsst/blob/main/python/lsst/obs/lsst/rawFormatter.py .

      I am happy to attempt to write and PR this but might need minor assistance.

       

        Attachments

          Issue Links

            Activity

            Hide
            wittgen Matthias Wittgen added a comment -

            New pull request from  lsst/afw https://github.com/lsst/afw/pull/655

            Show
            wittgen Matthias Wittgen added a comment - New pull request from  lsst/afw https://github.com/lsst/afw/pull/655
            Hide
            daniel.weatherill Daniel Weatherill added a comment -

            PR for the obs_lsst part here https://github.com/lsst/obs_lsst/pull/423

            Show
            daniel.weatherill Daniel Weatherill added a comment - PR for the obs_lsst part here https://github.com/lsst/obs_lsst/pull/423
            Hide
            tjenness Tim Jenness added a comment -

            The obs_lsst part looks good to me. Minor comments on the PR.

            Show
            tjenness Tim Jenness added a comment - The obs_lsst part looks good to me. Minor comments on the PR.
            Hide
            daniel.weatherill Daniel Weatherill added a comment -

            addressed github PR comments in the `obs_lsst` and `afw` parts of this issue. Will rebase properly and merge

            Show
            daniel.weatherill Daniel Weatherill added a comment - addressed github PR comments in the `obs_lsst` and `afw` parts of this issue. Will rebase properly and merge
            Hide
            mfisherlevine Merlin Fisher-Levine added a comment -

            It looks like both of these branches have merged. Should we close this ticket as "Done"?

            Show
            mfisherlevine Merlin Fisher-Levine added a comment - It looks like both of these branches have merged. Should we close this ticket as "Done"?

              People

              Assignee:
              daniel.weatherill Daniel Weatherill
              Reporter:
              daniel.weatherill Daniel Weatherill
              Reviewers:
              Matthias Wittgen
              Watchers:
              Daniel Weatherill, Matthias Wittgen, Merlin Fisher-Levine, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins Builds

                  No builds found.