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

Refactor ComputeExposureSummaryStats to allow fine-grained updates

    XMLWordPrintable

    Details

      Description

      This a spin-off from DM-35207, with the goal of making parallel work on that ticket and DM-37411 easier.

        Attachments

          Issue Links

            Activity

            Hide
            jbosch Jim Bosch added a comment -

            Lauren MacArthur, here are the changes I think I'll need for DM-35207 that would probably conflict with yours on DM-37411. Jenkins is running, so I can't promise that there will be no umprompted changes yet, and the timeline for the review is entirely up to you - I'm happy to start just basing my DM-35207 branches on these branches now and rebasing as needed.

            PRs:

            Show
            jbosch Jim Bosch added a comment - Lauren MacArthur , here are the changes I think I'll need for DM-35207 that would probably conflict with yours on DM-37411 . Jenkins is running , so I can't promise that there will be no umprompted changes yet, and the timeline for the review is entirely up to you - I'm happy to start just basing my DM-35207 branches on these branches now and rebasing as needed. PRs: https://github.com/lsst/afw/pull/670 https://github.com/lsst/pipe_tasks/pull/738
            Hide
            lauren Lauren MacArthur added a comment -

            I've left a few initial comments on the PRs.  I will give it a slightly deeper dive over the next day or two, but so far this looks great.

            Show
            lauren Lauren MacArthur added a comment - I've left a few initial comments on the PRs.  I will give it a slightly deeper dive over the next day or two, but so far this looks great.
            Hide
            jbosch Jim Bosch added a comment -

            I've pushed a few more commits throughout the course of the day, mostly to get Jenkins through, and it's now through at least Linux. So I think it's now ready for a final look, and I'll start putting any further changes onto DM-35207 branches instead.

            Show
            jbosch Jim Bosch added a comment - I've pushed a few more commits throughout the course of the day, mostly to get Jenkins through, and it's now through at least Linux. So I think it's now ready for a final look, and I'll start putting any further changes onto DM-35207 branches instead.
            Hide
            lauren Lauren MacArthur added a comment -

            I left a few very minor comments on the PRs, but this all looks good to me and will indeed make further use much easier.  Thanks!

            Show
            lauren Lauren MacArthur added a comment - I left a few very minor comments on the PRs, but this all looks good to me and will indeed make further use much easier.  Thanks!
            Hide
            lauren Lauren MacArthur added a comment -

            Oh, and don't forget to squash your "Will squash before merge." commits 

            Show
            lauren Lauren MacArthur added a comment - Oh, and don't forget to squash your "Will squash before merge." commits 

              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:

                  Jenkins

                  No builds found.