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

The way the observation date is translated by obs_cfht breaks the defect registry.

    Details

    • Story Points:
      1
    • Team:
      Alert Production

      Description

      The fields of the observation date and time as stored in the CFHT-LS headers are not padded with zeros. This makes the sqlite DATETIME constructor return a NULL string when the observation time is < 10hrs.

      The fix is to force the fields to be padded when the metadata from the input files are ingested.

        Attachments

          Activity

          Hide
          krughoff Simon Krughoff added a comment -

          Russell, can you take this? It is very small (3 lines).

          Show
          krughoff Simon Krughoff added a comment - Russell, can you take this? It is very small (3 lines).
          Hide
          rowen Russell Owen added a comment -

          This looks fine to me (though a unit test would be a nice addition).

          Show
          rowen Russell Owen added a comment - This looks fine to me (though a unit test would be a nice addition).
          Hide
          krughoff Simon Krughoff added a comment -

          I'm confused. Are you saying there is a problem in building the SDSS defect registry. I made a fix in obs_cfht, but if there is a problem with building SDSS defects for the same reason, it should go further up the stack.

          Show
          krughoff Simon Krughoff added a comment - I'm confused. Are you saying there is a problem in building the SDSS defect registry. I made a fix in obs_cfht, but if there is a problem with building SDSS defects for the same reason, it should go further up the stack.

            People

            • Assignee:
              krughoff Simon Krughoff
              Reporter:
              krughoff Simon Krughoff
              Reviewers:
              Russell Owen
              Watchers:
              Dominique Boutigny, Russell Owen, Simon Krughoff, Yusra AlSayyad
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Summary Panel

                  Time Tracking

                  Estimated:
                  Original Estimate - 1 hour
                  1h
                  Remaining:
                  Remaining Estimate - 1 hour
                  1h
                  Logged:
                  Time Spent - Not Specified
                  Not Specified