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

Move non-generic characterize/calibrate configs out of obs_lsst/config

    XMLWordPrintable

    Details

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

      Description

      While reviewing DM-32141, we noticed that the base-level obs_lsst configs have a mix of HSC- and simulation-specific values. In particular, the refcats in calibrate and characterizeImage are `cal_ref_cat`, which is the old "default" name and was used for the simulations, but is wrong for everything else. With DM-27013 merged, the refcat defaults for those tasks are now correct for everything except the simulations, so we shouldn't be overriding things unless we need to.

      This ticket is to move the simulation-specific overrides out of those files and into the imsim/phosim configs, and also to cleanup similar configs in ap_pipe post-DM-31063.

        Attachments

          Issue Links

            Activity

            Parejkoj John Parejko created issue -
            Parejkoj John Parejko made changes -
            Field Original Value New Value
            Link This issue is triggered by DM-32141 [ DM-32141 ]
            Parejkoj John Parejko made changes -
            Link This issue relates to DM-31063 [ DM-31063 ]
            Parejkoj John Parejko made changes -
            Link This issue relates to DM-27013 [ DM-27013 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-30506 [ 510172 ]
            Parejkoj John Parejko made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            Parejkoj John Parejko made changes -
            Reviewers Merlin Fisher-Levine, Nate Lust [ mfisherlevine, nlust ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            mfisherlevine Merlin Fisher-Levine made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            nlust Nate Lust made changes -
            Watchers Erik Dennihy, Ian Sullivan, John Parejko, Merlin Fisher-Levine, Nate Lust [ Erik Dennihy, Ian Sullivan, John Parejko, Merlin Fisher-Levine, Nate Lust ] Erik Dennihy, Ian Sullivan, John Parejko, Merlin Fisher-Levine [ Erik Dennihy, Ian Sullivan, John Parejko, Merlin Fisher-Levine ]
            Parejkoj John Parejko made changes -
            Link This issue relates to DM-34768 [ DM-34768 ]
            Parejkoj John Parejko made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]

              People

              Assignee:
              Parejkoj John Parejko
              Reporter:
              Parejkoj John Parejko
              Reviewers:
              Merlin Fisher-Levine, Nate Lust
              Watchers:
              Erik Dennihy, Ian Sullivan, John Parejko, Merlin Fisher-Levine
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.