# 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:
• 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.

#### Activity

Hide
Matthias Wittgen added a comment -

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

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

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

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

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

Show
Tim Jenness added a comment - The obs_lsst part looks good to me. Minor comments on the PR.
Hide
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 added a comment - addressed github PR comments in the obs_lsst and afw parts of this issue. Will rebase properly and merge
Hide
Merlin Fisher-Levine added a comment -

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

Show
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
Reporter:
Daniel Weatherill
Reviewers:
Matthias Wittgen
Watchers:
Daniel Weatherill, Matthias Wittgen, Merlin Fisher-Levine, Tim Jenness