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

Update registry schema (and documentation thereof) following review

    Details

    • Story Points:
      2
    • Sprint:
      BG3_F18_08, BG3_F18_09
    • Team:
      Data Release Production

      Description

      Make adjustments to the schema to incorporate feedback from RFC-484 and the associated review.

      Should also sync the technote with the schema YAML on daf_butler master (only very minor and structural changes so far).

       

        Attachments

          Issue Links

            Activity

            Hide
            jbosch Jim Bosch added a comment -

            Some concrete changes to remember to do (mostly from Russell Owen and Pim Schellart [X]):

            • Drop UNIQUE constraint from Sensor.name
            • Section 2.3: “nnames” -> “names”, "None null" -> "NULL"
            • page 6: “...potentially entries one or more metadata…” insert “in” before “one or more”
            • Execution table description "A unique autoincrement field used as part of the primary key for Execution.” (it is the primary key, not part of it)
            • “Run is specialization of Execution” -> “Run is a specialization of Execution”.

            More general questions:

            • Do we need the Datasets a Run points at to have DataUnits or be a part of Collections? What about Configs (or are those in the pipeline_id Dataset?)
            • Need to make sure all angle fields specify units.
            Show
            jbosch Jim Bosch added a comment - Some concrete changes to remember to do (mostly from Russell Owen and Pim Schellart [X] ): Drop UNIQUE constraint from Sensor.name Section 2.3: “nnames” -> “names”, "None null" -> "NULL" page 6: “...potentially entries one or more metadata…” insert “in” before “one or more” Execution table description "A unique autoincrement field used as  part  of the primary key for Execution.” (it is the primary key, not part of it) “Run is specialization of Execution” -> “Run is a specialization of Execution”. More general questions: Do we need the Datasets a Run points at to have DataUnits or be a part of Collections? What about Configs (or are those in the pipeline_id Dataset?) Need to make sure all angle fields specify units.
            Hide
            pschella Pim Schellart [X] (Inactive) added a comment -

            Units and reference frames.

            Show
            pschella Pim Schellart [X] (Inactive) added a comment - Units and reference frames.
            Hide
            jbosch Jim Bosch added a comment -

            Should add ra, dec to Tract table.

            Show
            jbosch Jim Bosch added a comment - Should add ra, dec to Tract table.
            Hide
            jbosch Jim Bosch added a comment -

            Changes in daf_butler, skymap (trivial), and dmtn-073.  I'm not sure the Python code in the technote really needs to be reviewed, but of course you're welcome to take a look (though looking at the diff with what used to be there is probably even less useful).

             

            Show
            jbosch Jim Bosch added a comment - Changes in daf_butler, skymap (trivial), and dmtn-073.  I'm not sure the Python code in the technote really needs to be reviewed, but of course you're welcome to take a look (though looking at the diff with what used to be there is probably even less useful).  
            Hide
            tjenness Tim Jenness added a comment -

            Looks fine. I discovered a daf_base dependency that I hadn't noticed before that concerns me.

            Show
            tjenness Tim Jenness added a comment - Looks fine. I discovered a daf_base dependency that I hadn't noticed before that concerns me.

              People

              • Assignee:
                jbosch Jim Bosch
                Reporter:
                jbosch Jim Bosch
                Reviewers:
                Tim Jenness
                Watchers:
                Jim Bosch, Pim Schellart [X] (Inactive), Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel