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

Type of IngestIndexedReferenceTask_config wrong in obs_ paf files

    XMLWordPrintable

    Details

      Description

      In DM-6651 I moved the new HTM indexed reference catalog code from pipe_tasks to meas_algorithms, but didn't do a complete job. The type of IngestIndexedReferenceTask_config in obs_ paf files still must be updated.

        Attachments

          Issue Links

            Activity

            Hide
            price Paul Price added a comment -

            I suggest moving the definition to daf_butlerUtils.

            Show
            price Paul Price added a comment - I suggest moving the definition to daf_butlerUtils.
            Hide
            rowen Russell Owen added a comment -

            Paul Price I agree

            Show
            rowen Russell Owen added a comment - Paul Price I agree
            Hide
            rowen Russell Owen added a comment -

            I added a new standard dataset type IngestIndexedReferenceTask_config to datasets.yaml in daf_butlerUtils and removed the two instances I found in existing obs_ packages (one in obs_test which was correct, and one in obs_lsstSim which I had not yet updated for new location of {{IngestIndexedReferenceTask).

            For proof that the new entry is correct I ran testPhotoCal.py in pipe_tasks tickets/DM-6325, which uses the task config in question.

            Show
            rowen Russell Owen added a comment - I added a new standard dataset type IngestIndexedReferenceTask_config to datasets.yaml in daf_butlerUtils and removed the two instances I found in existing obs_ packages (one in obs_test which was correct, and one in obs_lsstSim which I had not yet updated for new location of {{ IngestIndexedReferenceTask ). For proof that the new entry is correct I ran testPhotoCal.py in pipe_tasks tickets/ DM-6325 , which uses the task config in question.
            Hide
            reiss David Reiss added a comment -

            This all looks fine to me.

            Show
            reiss David Reiss added a comment - This all looks fine to me.

              People

              Assignee:
              rowen Russell Owen
              Reporter:
              rowen Russell Owen
              Reviewers:
              David Reiss
              Watchers:
              David Reiss, Paul Price, Russell Owen, Simon Krughoff
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.