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

Replace use of image <<= with [:] in python code

    XMLWordPrintable

    Details

    • Story Points:
      2
    • Team:
      Alert Production

      Description

      Replace all use of the afw image pixel copy operator <<= with [:] in Python code.

      See DM-4102 for the C++ version. These can be done independently.

        Attachments

          Issue Links

            Activity

            Hide
            rowen Russell Owen added a comment - - edited

            Implemented on branch tickets/DM-4100 in:

            • afw (python only)
            • display_ds9
            • ip_diffIm (python only)
            • ip_isr
            • obs_lsstSim
            • obs_subaru
            • obs_test
            • meas_algorithms (python only)
            • meas_astrom
            • pipe_tasks
            Show
            rowen Russell Owen added a comment - - edited Implemented on branch tickets/ DM-4100 in: afw (python only) display_ds9 ip_diffIm (python only) ip_isr obs_lsstSim obs_subaru obs_test meas_algorithms (python only) meas_astrom pipe_tasks
            Hide
            krughoff Simon Krughoff added a comment -

            I didn't want to create a pull request for each of these packages so I'm just going to put my comments here.

            obs_lsstSim(line 127): python/lsst/obs/lsstSim/processEimage.py missing space before = sign.

            Looks good otherwise.

            Show
            krughoff Simon Krughoff added a comment - I didn't want to create a pull request for each of these packages so I'm just going to put my comments here. obs_lsstSim(line 127): python/lsst/obs/lsstSim/processEimage.py missing space before = sign. Looks good otherwise.
            Hide
            rowen Russell Owen added a comment -

            Thanks! Merged (with the fix for the problem you noted) and pushed.

            Show
            rowen Russell Owen added a comment - Thanks! Merged (with the fix for the problem you noted) and pushed.

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              rowen Russell Owen
              Reviewers:
              Simon Krughoff
              Watchers:
              Russell Owen, Simon Krughoff
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  CI Builds

                  No builds found.