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

Remove Gen2 usage from meas_base

    XMLWordPrintable

Details

    • 3
    • Architecture
    • No

    Description

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

      Attachments

        Issue Links

          Activity

            tjenness Tim Jenness added a comment -

            lauren 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.

            tjenness Tim Jenness added a comment - lauren 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.

            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...)?

            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...)?
            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 erykoff merges DM-35670 first (which has the same fix in it).

            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 erykoff merges DM-35670 first (which has the same fix in it).

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

            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

              tjenness Tim Jenness
              tjenness Tim Jenness
              Lauren MacArthur
              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.