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

Remove Gen2 usage from meas_base

    XMLWordPrintable

    Details

    • Story Points:
      3
    • Team:
      Architecture
    • Urgent?:
      No

      Description

      meas_base uses a gen2/3 compatibility function from coadd_utils and also has runDataRef methods.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            Lauren MacArthur you seem to have worked with this code a bit so can you please take a look at the changes? I came to the conclusion that the reference subtask is not used any more and ci_hsc_gen3 seems to run fine without it.

            Show
            tjenness Tim Jenness added a comment - Lauren MacArthur you seem to have worked with this code a bit so can you please take a look at the changes? I came to the conclusion that the reference subtask is not used any more and ci_hsc_gen3 seems to run fine without it.
            Hide
            lauren Lauren MacArthur added a comment -

            I just left a bunch of comments on the meas_base PR. Did you want me to look at pipe_tasks too (I don't see a PR...)?

            Show
            lauren Lauren MacArthur added a comment - I just left a bunch of comments on the meas_base PR. Did you want me to look at pipe_tasks too (I don't see a PR...)?
            Hide
            tjenness Tim Jenness added a comment - - edited

            The pipe_tasks change is just a small removal of the usage of PerTractCcdDataIdContainer. Whether I merge it or not depends on whether Eli Rykoff merges DM-35670 first (which has the same fix in it).

            Show
            tjenness Tim Jenness added a comment - - edited The pipe_tasks change is just a small removal of the usage of PerTractCcdDataIdContainer . Whether I merge it or not depends on whether Eli Rykoff merges DM-35670 first (which has the same fix in it).
            Hide
            lauren Lauren MacArthur added a comment -

            Then I guess my work here is done!   Pending successful Jenkins + ci_hsc + ci_imsim, good to go.

            Show
            lauren Lauren MacArthur added a comment - Then I guess my work here is done!   Pending successful Jenkins + ci_hsc + ci_imsim, good to go.

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Lauren MacArthur
              Watchers:
              Eli Rykoff, Lauren MacArthur, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.