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

Deprecate VisitInfo.getExposureId()

    XMLWordPrintable

    Details

      Description

      exposureId in VisitInfo is not actually a visit-level value, but a ccd-specific value. This requires deprecating `getExposureId()` in visitInfo, so that we can remove it, and replacing any uses of that method with the replacement from DM-29955.

        Attachments

          Issue Links

          blocks

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-32138 Remove VisitInfo.exposureId

          • To Do

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-31706 Do Release 24.0.0 of Science Pipelines

          • Done
          is blocked by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-29955 Add ExposureInfo id getter (replacement for VisitInfo.getExposureId())

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-13944 add id to VisitInfo

          • Done
          is duplicated by

          Bug - A problem which impairs or prevents the functions of the product. DM-13738 Remove exposureId from visitInfo

          • Invalid
          is triggered by

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-459 Remove exposureId from VisitInfo and add visitId

          • Implemented

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            krzys Krzysztof Findeisen added a comment -

            From DM with John Parejko:

            So, the deprecation replacement message would say if you want a full focal plane identifier, use visitInfo.id; if you want a detector level identifier, use exposureInfo.id.

            Show
            krzys Krzysztof Findeisen added a comment - From DM with John Parejko : So, the deprecation replacement message would say if you want a full focal plane identifier, use visitInfo.id; if you want a detector level identifier, use exposureInfo.id.
            Hide
            Permalink
            krzys Krzysztof Findeisen added a comment -

            Thanks for agreeing to review this, Spencer Nelson. As usual not all of the PRs were picked up by Jira:

            Show
            krzys Krzysztof Findeisen added a comment - Thanks for agreeing to review this, Spencer Nelson . As usual not all of the PRs were picked up by Jira: geom#35 afw#614 obs_base#396 obs_test#87 obs_decam#207 ip_isr#202 pipe_tasks#598 cp_pipe#108 ap_association#140

              People

              Assignee:
              krzys Krzysztof Findeisen
              Reporter:
              Parejkoj John Parejko
              Reviewers:
              Spencer Nelson
              Watchers:
              Jim Bosch, John Parejko, Krzysztof Findeisen, Russell Owen, Spencer Nelson
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.