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

Deprecate characterizeImage refcat hooks

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: pipe_tasks
    • Labels:
      None

      Description

      The characterizeImage refObjLoader config does nothing, so we should deprecate and remove it. It's resulted in some confusion when configuring the LATISS pipeline.

      On this ticket, I'll also remove any references to it in our own configs.

        Attachments

          Issue Links

          blocks

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-34769 Remove characterizeImage refObjLoader hooks

          • To Do
          is duplicated by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-19252 Deprecate and stop using characterizeImage.refObjLoader

          • Invalid
          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-32141 Get AuxTel SFM reductions working end-to-end

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-34752 Move non-generic characterize/calibrate configs out of obs_lsst/config

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-19252 Deprecate and stop using characterizeImage.refObjLoader

          • Invalid

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            Parejkoj John Parejko added a comment -

            Jenkins: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/36572/pipeline

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

            Jim Bosch: do you mind reviewing this deprecation, since you had some knowledge of this code in the past? I believe I've removed all references to the things I deprecated, so there shouldn't be any warnings triggered by this in stack code.

            Not all the PRs were showing up in Jira for me, so here they are:

            Show
            Parejkoj John Parejko added a comment - Jim Bosch : do you mind reviewing this deprecation, since you had some knowledge of this code in the past? I believe I've removed all references to the things I deprecated, so there shouldn't be any warnings triggered by this in stack code. Not all the PRs were showing up in Jira for me, so here they are: https://github.com/lsst/pipe_tasks/pull/679 https://github.com/lsst/obs_lsst/pull/399 https://github.com/lsst/obs_cfht/pull/100 https://github.com/lsst/obs_decam/pull/220 https://github.com/lsst/ap_pipe/pull/109
            Hide
            Permalink
            jbosch Jim Bosch added a comment -

            Looks good. Only PR comment is basically, "you can remove even more if you'd like."

            Show
            jbosch Jim Bosch added a comment - Looks good. Only PR comment is basically, "you can remove even more if you'd like."

              People

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

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.