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

Make Detector.transform and Camera.transform support lists of points

    XMLWordPrintable

    Details

    • Story Points:
      5
    • Epic Link:
    • Sprint:
      AP S18-2, AP S18-3
    • Team:
      Alert Production

      Description

      Implement RFC-392: make Detector.transform and Camera.transform support lists of points (or single points) and delete the CameraPoint class. Removing the CameraPoint class also requires a minor change to Camera.findDetectors and findDetectorsList.

        Attachments

          Issue Links

            Activity

            rowen Russell Owen created issue -
            rowen Russell Owen made changes -
            Field Original Value New Value
            Link This issue is triggered by RFC-392 [ RFC-392 ]
            rowen Russell Owen made changes -
            Sprint AP S18-2 [ 677 ]
            rowen Russell Owen made changes -
            Rank Ranked higher
            swinbank John Swinbank made changes -
            Epic Link DM-9679 [ 30784 ]
            swinbank John Swinbank made changes -
            Sprint AP S18-2 [ 677 ] AP S18-2, AP S18-3 [ 677, 683 ]
            rowen Russell Owen made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            rowen Russell Owen made changes -
            Description Implement RFC-392: remove TransformMap.transform, Camera.transform and Detector.transform and the CameraPoint class. Modify existing code to use a {{TransformPoint2ToPoint2}} obtained from a {{Camera}} or {{AmpInfo}} object. Implement RFC-392: remove TransformMap.transform, Camera.transform and Detector.transform and delete the CameraPoint class. Modify existing code to use a {{TransformPoint2ToPoint2}} obtained from {{Camera.getTransform}} or {{Detector.getTransform}}.

            This also requires a few changes to Camera.findDetectors (change the single camera point argument to a point and a camera sys) and findDetectorsList (the first argument will be a list of points, not camera points).
            rowen Russell Owen made changes -
            Component/s ip_isr [ 10730 ]
            rowen Russell Owen made changes -
            Component/s jointcal [ 13411 ]
            Component/s meas_mosaic [ 10742 ]
            Component/s obs_sdss [ 10763 ]
            rowen Russell Owen made changes -
            Reviewers Paul Price [ price ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            rowen Russell Owen made changes -
            Component/s meas_base [ 10750 ]
            price Paul Price made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            rowen Russell Owen made changes -
            Status Reviewed [ 10101 ] In Review [ 10004 ]
            rowen Russell Owen made changes -
            Description Implement RFC-392: remove TransformMap.transform, Camera.transform and Detector.transform and delete the CameraPoint class. Modify existing code to use a {{TransformPoint2ToPoint2}} obtained from {{Camera.getTransform}} or {{Detector.getTransform}}.

            This also requires a few changes to Camera.findDetectors (change the single camera point argument to a point and a camera sys) and findDetectorsList (the first argument will be a list of points, not camera points).
            Implement RFC-392: make Detector.transform and Camera.transform support lists of points (or single points) and delete the CameraPoint class. Removing the CameraPoint class also requires a minor change to Camera.findDetectors and findDetectorsList.
            Summary Remove TransformMap.transform, Camera.transform and Detector.transform Make Detector.transform and Camera.transform support lists of points
            rowen Russell Owen made changes -
            Component/s pipe_tasks [ 10726 ]
            price Paul Price made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            rowen Russell Owen made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]

              People

              Assignee:
              rowen Russell Owen
              Reporter:
              rowen Russell Owen
              Reviewers:
              Paul Price
              Watchers:
              Krzysztof Findeisen, Paul Price, Russell Owen
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  CI Builds

                  No builds found.