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

Add descriptions to CcdVisit and Visit Tables in sdm_schemas

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      CcdVisit and Visit schemas need descriptions and units.

      Needs backporting to v23.0.x imsim.yaml such that:
      v23.0.x/imsim.yaml == main/dp02_dc2.yaml

      They can be copied from the baseline schema:
      https://github.com/lsst/sdm_schemas/blob/main/yml/baselineSchema.yaml

        Attachments

          Activity

          Hide
          csaunder Clare Saunders added a comment -

          Successful Jenkins run, though I'm not sure if this tests the descriptions and units: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/36832/pipeline

          Show
          csaunder Clare Saunders added a comment - Successful Jenkins run, though I'm not sure if this tests the descriptions and units: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/36832/pipeline
          Hide
          csaunder Clare Saunders added a comment -

          Hi Colin, Yusra said you'd volunteered to review this. Thanks!

          Show
          csaunder Clare Saunders added a comment - Hi Colin, Yusra said you'd volunteered to review this. Thanks!
          Hide
          ctslater Colin Slater added a comment -

          Thanks a lot for taking care of this, these are all really helpful. I've left a handful of unusually-picky comments on the PR; since I expect "novice" LSST data product users to be the main audience for these descriptions, I think it's worth heading off any potential confusion even if it adds a bit of redundancy.

          Show
          ctslater Colin Slater added a comment - Thanks a lot for taking care of this, these are all really helpful. I've left a handful of unusually-picky comments on the PR; since I expect "novice" LSST data product users to be the main audience for these descriptions, I think it's worth heading off any potential confusion even if it adds a bit of redundancy.

            People

            Assignee:
            csaunder Clare Saunders
            Reporter:
            yusra Yusra AlSayyad
            Reviewers:
            Colin Slater
            Watchers:
            Clare Saunders, Colin Slater, John Parejko, Yusra AlSayyad
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.