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

Fix logic of the doVignette option in isrTask

    XMLWordPrintable

    Details

    • Story Points:
      4
    • Epic Link:
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      As was noted in DM-29280 (and directly quoting from this comment):

      Now, in isrTask, there is a doVignette config said to "Apply vignetting parameters", but I am very puzzled by this option. It fires only after all the other ISR operations have fired (so none of them would know about any effect this operation has). If True, it calls the VignetteTask in ip_isr, but that task returns None unless the config parameter doWriteVignettePolygon is True, in which case it returns a polygon. In other words, if doWriteVignettePolygon is False, the call to this function is a no-op, so why make it at all? Also, the function gets an exposure passed in as a required parameter, but as far as I can tell, it is not used (nor updated) in the function. If doWriteVignettePolygon is indeed True, then the validPolygon is attached to the exposure (but it doesn't seem that any mask bits are set...but that might be as intended).

      This all seems to be a bit of a mess as one can have a validPolygon representing the vignetted region attached to an exposure, but this does not imply a mask bit will be set...

      This purpose of this ticket is to clean up the logic surrounding the validPolygon setting and masking, and usage therein, of the vignetted region of a detector.

        Attachments

          Issue Links

            Activity

            Hide
            lauren Lauren MacArthur added a comment -

            I've been playing around with this, so will take it over (for now...if and up until I get stuck!)

            Show
            lauren Lauren MacArthur added a comment - I've been playing around with this, so will take it over (for now...if and up until I get stuck!)
            Hide
            plazas Andrés Alejandro Plazas Malagón added a comment -

            It makes sense to me. I just left some minor comments.

            Show
            plazas Andrés Alejandro Plazas Malagón added a comment - It makes sense to me. I just left some minor comments.

              People

              Assignee:
              czw Christopher Waters
              Reporter:
              lauren Lauren MacArthur
              Reviewers:
              Andrés Alejandro Plazas Malagón
              Watchers:
              Andrés Alejandro Plazas Malagón, Christopher Waters, Jim Bosch, Lauren MacArthur
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.