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

    XMLWordPrintable

    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

          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-11163 Always create VisitInfo from metadata when available

          • Done

            Activity

            Ascending order - Click to sort in descending order
            No builds found.
            mfisherlevine Merlin Fisher-Levine created issue -
            Hide
            Permalink
            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
            Permalink
            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
            Permalink
            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:

                  Jenkins

                  No builds found.