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

            Activity

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