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

Add MJD times to CcdVisit/Visit tables

    XMLWordPrintable

    Details

    • Team:
      DM Science
    • Urgent?:
      No

      Description

      CcdVisit and Visit have exposure start times and midpoint times available as timestamp types, but to provide users an easy way to compute time deltas without the risk of leap second or timezone issues, it would help to have MJD versions as well.

        Attachments

          Activity

          Hide
          ctslater Colin Slater added a comment -

          Hi Clare Saunders, do you have time to review this small addition the Visit and CcdVisit tables? I've also adjusted Visit obsStart to be consistent with the value in CcdVisit; let me know if I misunderstood the logic there,

          Show
          ctslater Colin Slater added a comment - Hi Clare Saunders , do you have time to review this small addition the Visit and CcdVisit tables? I've also adjusted Visit obsStart to be consistent with the value in CcdVisit; let me know if I misunderstood the logic there,
          Hide
          csaunder Clare Saunders added a comment -

          Hi Colin Slater, I'll take a look at this tomorrow. Please ping me if you don't see a review by the end of tomorrow, as I am in the middle of moving and might loose track of things.

          Show
          csaunder Clare Saunders added a comment - Hi Colin Slater , I'll take a look at this tomorrow. Please ping me if you don't see a review by the end of tomorrow, as I am in the middle of moving and might loose track of things.
          Hide
          csaunder Clare Saunders added a comment -

          This looks good to me. Thanks for fixing the inconsistency between the CcdVisit and Visit tables. Is there a Jenkins run for the ticket?

          Show
          csaunder Clare Saunders added a comment - This looks good to me. Thanks for fixing the inconsistency between the CcdVisit and Visit tables. Is there a Jenkins run for the ticket?
          Hide
          ctslater Colin Slater added a comment -

          Thanks for the review, Jenkins run is https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/36276/pipeline for reference. Merged to main.

          Show
          ctslater Colin Slater added a comment - Thanks for the review, Jenkins run is https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/36276/pipeline for reference. Merged to main.
          Hide
          yusra Yusra AlSayyad added a comment -

          I remember we approved this backport at the last meeting, so I'm going to add the label now. 

          Show
          yusra Yusra AlSayyad added a comment - I remember we approved this backport at the last meeting, so I'm going to add the label now. 

            People

            Assignee:
            ctslater Colin Slater
            Reporter:
            ctslater Colin Slater
            Reviewers:
            Clare Saunders
            Watchers:
            Clare Saunders, Colin Slater, Yusra AlSayyad
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.