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

Document schema naming conventions

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Labels:
      None

      Description

      Schema naming conventions changed from "."-separated with no case consistency to "_" and camelCase with the introduction of meas_base.  I remember writing the new conventions down somewhere, but the first two places I looked:

       - afw::table::Schema class docs

       - afw::table overview page in Doxygen

      ...document the old convention.

       

      Fix those, ideally by locating the original text (maybe Confluence somewhere?) and transferring it to those locations.

       

        Attachments

          Issue Links

            Activity

            Hide
            jbosch Jim Bosch added a comment -

            Lauren MacArthur, since you brought this to my attention, you get to review it .

            Show
            jbosch Jim Bosch added a comment - Lauren MacArthur , since you brought this to my attention, you get to review it .
            Hide
            lauren Lauren MacArthur added a comment - - edited

            Looks good.  A few minor comments on the PR, and I think the docs in afw/include/lsst/afw/table/Field.h also need updating.

            Show
            lauren Lauren MacArthur added a comment - - edited Looks good.  A few minor comments on the PR, and I think the docs in  afw/include/lsst/afw/table/Field.h also need updating.

              People

              • Assignee:
                jbosch Jim Bosch
                Reporter:
                jbosch Jim Bosch
                Reviewers:
                Lauren MacArthur
                Watchers:
                Jim Bosch, Lauren MacArthur
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: