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

obs_cfht tests fall over due to -9999.9 for TELEAZ for calibs

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: obs_cfht
    • Labels:
      None

      Description

      After DM-11163 (and its related tickets), visitInfos are created for calibs (we need darkTimes and expTimes etc).

      However, some of the metadata in the testdata_cfht contains values which make makeRawVisitInfo unhappy. Luckily, the megacamMapper had a _standardizeDetrend method for stripping out this which upset things downstream.

      This ticket just adds the TELEAZ and TELEALT keywords to those removed. This should only be called for biases, and therefore not upset anything else.

        Attachments

          Issue Links

            Activity

            mfisherlevine Merlin Fisher-Levine created issue -
            Hide
            rhl Robert Lupton added a comment -

            Well, biases/darks/etc.

            This is functionality that should be moved to obs_base once Jenkins is happy and the story points are available.

            Show
            rhl Robert Lupton added a comment - Well, biases/darks/etc. This is functionality that should be moved to obs_base once Jenkins is happy and the story points are available.
            mfisherlevine Merlin Fisher-Levine made changes -
            Field Original Value New Value
            Status To Do [ 10001 ] In Progress [ 3 ]
            mfisherlevine Merlin Fisher-Levine made changes -
            Reviewers John Swinbank [ swinbank ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            Hide
            swinbank John Swinbank added a comment -

            Looks good. Thank you!

            Show
            swinbank John Swinbank added a comment - Looks good. Thank you!
            swinbank John Swinbank made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            Hide
            mfisherlevine Merlin Fisher-Levine added a comment -

            Merged.

            Show
            mfisherlevine Merlin Fisher-Levine added a comment - Merged.
            mfisherlevine Merlin Fisher-Levine made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            swinbank John Swinbank made changes -
            Epic Link DM-10382 [ 32101 ]
            swinbank John Swinbank made changes -
            Link This issue relates to DM-11163 [ DM-11163 ]
            swinbank John Swinbank made changes -
            Story Points 1
            Team Data Release Production [ 10301 ]

              People

              • Assignee:
                mfisherlevine Merlin Fisher-Levine
                Reporter:
                mfisherlevine Merlin Fisher-Levine
                Reviewers:
                John Swinbank
                Watchers:
                John Swinbank, Merlin Fisher-Levine, Robert Lupton
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel