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.

    XMLWordPrintable

    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

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          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
          Permalink
          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
          Permalink
          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:

                Time Tracking

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

                  Jenkins

                  No builds found.