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

Make algorithm metadata writing FITS-compliant

    XMLWordPrintable

    Details

    • Story Points:
      8
    • Sprint:
      DRP F16-4, DRP F16-5
    • Team:
      Data Release Production

      Description

      FITS header keys should be all uppercase, and CFITSIO 3.39 (and possibly other versions > 3.36) enforce this. This breaks testCoadds.py, which currently expects algorithm metadata keys to be round-tripped exactly.

      See also https://community.lsst.org/t/fits-and-lowercase-header-keys/1184.

        Attachments

          Activity

          Hide
          jbosch Jim Bosch added a comment -

          I'm moving this back into To-Do, out of the sprint, and out of its current epic (I hope that's what John Swinbank would want me to do): I think this still needs to be done eventually, but it's both gotten harder and lower-priority as I've learned more about the problem (see the community post linked above).

          The existing branch contains only a workaround that allowed me to use CFITSIO 3.39, but apparently even newer versions have reverted the new strictness added in that version (which is what we're running afoul of here).

          Show
          jbosch Jim Bosch added a comment - I'm moving this back into To-Do, out of the sprint, and out of its current epic (I hope that's what John Swinbank would want me to do): I think this still needs to be done eventually, but it's both gotten harder and lower-priority as I've learned more about the problem (see the community post linked above). The existing branch contains only a workaround that allowed me to use CFITSIO 3.39, but apparently even newer versions have reverted the new strictness added in that version (which is what we're running afoul of here).
          Hide
          swinbank John Swinbank added a comment -

          Following up on the above to shift this not just out of the sprint but out of any epics scheduled for F16.

          Show
          swinbank John Swinbank added a comment - Following up on the above to shift this not just out of the sprint but out of any epics scheduled for F16.

            People

            Assignee:
            jbosch Jim Bosch
            Reporter:
            jbosch Jim Bosch
            Watchers:
            Jim Bosch, John Swinbank
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:

                Jenkins Builds

                No builds found.