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

Finish implementing YAML storage for PropertyList/PropertySet

    Details

    • Story Points:
      6
    • Team:
      Architecture

      Description

      Implement full round-trip read and write from YAML for PropertySet and PropertyList.

      Started here: https://github.com/lsst/daf_persistence/commit/6ed767db014fbcec7478ff7d616de3b0fc691b14

        Attachments

          Issue Links

            Activity

            Hide
            ktl Kian-Tat Lim added a comment -

            I'm not yet changing the obs_ packages to actually use this storage, nor removing the code to read/write boost::persistence storage.  If Jim Bosch wants, that could become one or two more tickets.

            Jenkins has finished on Linux: 28021.

            Show
            ktl Kian-Tat Lim added a comment - I'm not yet changing the obs_ packages to actually use this storage, nor removing the code to read/write boost::persistence storage.  If Jim Bosch wants, that could become one or two more tickets. Jenkins has finished on Linux: 28021.
            Hide
            rowen Russell Owen added a comment -

            Overall this looks good. My main requested change is to use TAI, not UTC, to save dates, primarily because TAI is better (no table of leap seconds is needed to understand it), but also because DateTime mis-handles UTC at leap seconds. I'd also suggested some documentation updates on github.

            Show
            rowen Russell Owen added a comment - Overall this looks good. My main requested change is to use TAI, not UTC, to save dates, primarily because TAI is better (no table of leap seconds is needed to understand it), but also because DateTime mis-handles UTC at leap seconds. I'd also suggested some documentation updates on github.
            Hide
            ktl Kian-Tat Lim added a comment -

            Changed to TAI, rebased on master, ran on Jenkins (past afw, until a known problem with meas_algorithms), merged to master.

            Show
            ktl Kian-Tat Lim added a comment - Changed to TAI, rebased on master, ran on Jenkins (past afw, until a known problem with meas_algorithms), merged to master.

              People

              • Assignee:
                ktl Kian-Tat Lim
                Reporter:
                jbosch Jim Bosch
                Reviewers:
                Russell Owen
                Watchers:
                Jim Bosch, Kian-Tat Lim, Russell Owen
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel