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

Update obs package use of filter.getId() in _computeCoaddExposureId

    XMLWordPrintable

    Details

    • Story Points:
      2
    • Urgent?:
      No

      Description

      This may be a "won't fix" because it only affects the gen2 CameraMappers, but lsst, subaru, decam, and cfht all use filter.getId() in their calculation of the singleFilter coadd exposure id. The new FilterLabel does not have a numeric Id, so these mappers will have to be changed. However, I do not know what the time ordering of gen2 removal vs. Filter removal is: if gen2 is removed first, then this won't matter at all.

      I did not change this on DM-27170 because I did not want to alter this calculation in gen2 without more discussion, and want to get that ticket finished up.

        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-27170 [ DM-27170 ]
            Parejkoj John Parejko made changes -
            Link This issue blocks DM-27177 [ DM-27177 ]
            sullivan Ian Sullivan made changes -
            Labels SciencePipelines
            krzys Krzysztof Findeisen made changes -
            Labels SciencePipelines SciencePipelines gen2-only
            Parejkoj John Parejko made changes -
            Watchers Ian Sullivan, Jim Bosch, John Parejko, Krzysztof Findeisen, Tim Jenness [ Ian Sullivan, Jim Bosch, John Parejko, Krzysztof Findeisen, Tim Jenness ] Ian Sullivan, Jim Bosch, John Parejko, Krzysztof Findeisen, Tim Jenness, Yusra AlSayyad [ Ian Sullivan, Jim Bosch, John Parejko, Krzysztof Findeisen, Tim Jenness, Yusra AlSayyad ]
            jbosch Jim Bosch made changes -
            Resolution Done [ 10000 ]
            Status To Do [ 10001 ] Won't Fix [ 10405 ]

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              Parejkoj John Parejko
              Watchers:
              Ian Sullivan, Jim Bosch, John Parejko, Krzysztof Findeisen, Lauren MacArthur, Tim Jenness, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.