Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-1766 Remove in-memory support of old-version afw::table objects
  3. DM-2535

Backwards compatibility for reading compound fields from FITS

    XMLWordPrintable

    Details

    • Type: Technical task
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Labels:
      None
    • Story Points:
      2
    • Sprint:
      Science Pipelines DM-S15-2, Science Pipelines DM-S15-3, Science Pipelines DM-S15-4
    • Team:
      Data Release Production

      Description

      Read old-style afw::table compound fields in as scalar fields, using the new FunctorKey conventions.

        Attachments

          Issue Links

            Activity

            Hide
            jbosch Jim Bosch added a comment -

            Russell Owen, this is the bulk of my changes to remove version=0 tables. It sits on top of changes from DM-2533 and DM-2534 in some packages, with more changes on DM-2536 yet to come. I'm getting them all reviewed separately, but I'll merge them all together, which is why you may see some unit tests skipped in what I've asked you to review - the skipping should be reverted in DM-2536. But even that's pretty rare.

            See the linked afw PR for a description of the changes. There are GitHub PRs already created for 7 packages: afw, meas_base, meas_algorithms, meas_astrom, meas_deblender, ip_diffim, and pipe_tasks. JIRA seems to be a bit slow at displaying them, but I think they'll turn up. I took some care to make sure these show the correct diffs; the branch the diff is relative to depends on whether there were changes on DM-2533 and/or DM-2534.

            Show
            jbosch Jim Bosch added a comment - Russell Owen , this is the bulk of my changes to remove version=0 tables. It sits on top of changes from DM-2533 and DM-2534 in some packages, with more changes on DM-2536 yet to come. I'm getting them all reviewed separately, but I'll merge them all together, which is why you may see some unit tests skipped in what I've asked you to review - the skipping should be reverted in DM-2536 . But even that's pretty rare. See the linked afw PR for a description of the changes. There are GitHub PRs already created for 7 packages: afw, meas_base, meas_algorithms, meas_astrom, meas_deblender, ip_diffim, and pipe_tasks. JIRA seems to be a bit slow at displaying them, but I think they'll turn up. I took some care to make sure these show the correct diffs; the branch the diff is relative to depends on whether there were changes on DM-2533 and/or DM-2534 .
            Hide
            rowen Russell Owen added a comment -

            Everything looks very good to me. Overall it appears to be a nice improvement. I did add a few minor requests and comments to the pull requests.

            Show
            rowen Russell Owen added a comment - Everything looks very good to me. Overall it appears to be a nice improvement. I did add a few minor requests and comments to the pull requests.
            Hide
            jbosch Jim Bosch added a comment -

            Merged to master with tickets/DM-1766 (where changes after review were rebased).

            Show
            jbosch Jim Bosch added a comment - Merged to master with tickets/ DM-1766 (where changes after review were rebased).

              People

              Assignee:
              jbosch Jim Bosch
              Reporter:
              jbosch Jim Bosch
              Reviewers:
              Russell Owen
              Watchers:
              Jim Bosch, Russell Owen
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: