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

Disable jointcal photometry in HSC via config

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: jointcal, obs_subaru
    • Labels:
      None
    • Story Points:
      1
    • Sprint:
      AP S22-1 (December)
    • Team:
      Alert Production
    • Urgent?:
      No

      Description

      DM-29615 originally disabled jointcal photometry in Gen3 only via the obs_subaru DRP pipeline definition. In the review it was quite reasonably suggested that we should disable this in the obs_subaru configs instead (which avoids a back-and-forther set-unset in config override and should turn it off in Gen2 as well as Gen3, which is good for consistency).

      Unfortunately that got tricky (in the form of unexpected jointcal test failures), so I declared it out of scope and just left jointcal photometry enabled in the merged version of that ticket.

      But it turns out that doesn't work, because of DM-29884. So on DM-29776 I'm going back to the original, Gen3-only DM-29615 solution, and on this ticket we can figure out how to replace that with a cleaner config-based solution.

        Attachments

          Issue Links

            Activity

            Show
            Parejkoj John Parejko added a comment - Jenkins: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/35705/pipeline
            Hide
            Parejkoj John Parejko added a comment -

            Lauren MacArthur: This is the change that you requested I do on DM-29008. I wanted to test it independently so I could get that ticket merged. Should be a trivial review.

            Show
            Parejkoj John Parejko added a comment - Lauren MacArthur : This is the change that you requested I do on DM-29008 . I wanted to test it independently so I could get that ticket merged. Should be a trivial review.
            Hide
            lauren Lauren MacArthur added a comment -

            LGTM.

            Show
            lauren Lauren MacArthur added a comment - LGTM.

              People

              Assignee:
              Parejkoj John Parejko
              Reporter:
              jbosch Jim Bosch
              Reviewers:
              Lauren MacArthur
              Watchers:
              Jim Bosch, John Parejko, Lauren MacArthur
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.