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

overscan improperly sets bounding boxes when leading/trailing columns are skipped

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ip_isr
    • Labels:
      None
    • Story Points:
      1
    • Team:
      Data Release Production

      Description

      Bounding box handling of `overscanNumLeadingColumnsToSkip` and `overscanNumTrailingColumnsToSkip` is incorrect.

        Attachments

          Issue Links

            Activity

            No builds found.
            czw Christopher Waters created issue -
            czw Christopher Waters made changes -
            Field Original Value New Value
            Component/s ip_isr [ 10730 ]
            Team Data Release Production [ 10301 ]
            Watchers Christopher Waters [ Christopher Waters ] Christopher Waters, Robert Lupton [ Christopher Waters, Robert Lupton ]
            Assignee Christopher Waters [ cwaters ]
            Description Bounding box handling of `overscanNumLeadingColumnsToSkip` and `overscanNumTrailingColumnsToSkip` is incorrect.
            czw Christopher Waters made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            czw Christopher Waters made changes -
            Story Points 1
            Hide
            rhl Robert Lupton added a comment -

            This ticket breaks ci_lsst, so I can't merge until this is merged to master. So it also blocks v17

            Show
            rhl Robert Lupton added a comment - This ticket breaks ci_lsst , so I can't merge until this is merged to master. So it also blocks v17
            rhl Robert Lupton made changes -
            Link This issue blocks DM-17376 [ DM-17376 ]
            tjenness Tim Jenness made changes -
            Link This issue blocks DM-17387 [ DM-17387 ]
            tjenness Tim Jenness made changes -
            Link This issue blocks DM-17376 [ DM-17376 ]
            tjenness Tim Jenness made changes -
            Watchers Christopher Waters, Robert Lupton [ Christopher Waters, Robert Lupton ] Christopher Waters, John Swinbank, Robert Lupton [ Christopher Waters, John Swinbank, Robert Lupton ]
            Hide
            krughoff Simon Krughoff added a comment - - edited

            Looks good.

            I have a couple of more or less philosophical comments.

            Show
            krughoff Simon Krughoff added a comment - - edited Looks good. I have a couple of more or less philosophical comments.
            krughoff Simon Krughoff made changes -
            Status In Progress [ 3 ] Reviewed [ 10101 ]
            czw Christopher Waters made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]

              People

              Assignee:
              czw Christopher Waters
              Reporter:
              czw Christopher Waters
              Watchers:
              Christopher Waters, John Swinbank, Robert Lupton, Simon Krughoff
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.