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

display.dot of ellipses fails with display_firefly when buffering

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: display_firefly
    • Labels:
      None
    • Story Points:
      2
    • Sprint:
      SUIT Sprint 2018-08
    • Team:
      Science User Interface

      Description

      When using afw.display with the Firefly backend, writing ellipses with buffering does not draw ellipses:
       

      with display.Buffering():
          for record in src256[::10]:
              display.dot(record.getShape(), record.getX(), record.getY())
      

      It works without the buffering.

        Attachments

          Issue Links

            Activity

            No builds found.
            shupe David Shupe created issue -
            shupe David Shupe made changes -
            Field Original Value New Value
            Risk Score 0
            Hide
            shupe David Shupe added a comment -

            NaN values in the catalog cause the problems. When not buffering, the regions are sent individually and the valid ellipses are drawn. When buffering is on, the regions are sent all at once and the NaN-shape ones cause all the regions to fail.

            Show
            shupe David Shupe added a comment - NaN values in the catalog cause the problems. When not buffering, the regions are sent individually and the valid ellipses are drawn. When buffering is on, the regions are sent all at once and the NaN-shape ones cause all the regions to fail.
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Link This issue is cloned by DM-15245 [ DM-15245 ]
            shupe David Shupe made changes -
            Watchers Cindy Wang, David Shupe, Trey Roby, Xiuqin Wu [ Cindy Wang, David Shupe, Trey Roby, Xiuqin Wu ] Cindy Wang, David Shupe, Robert Lupton, Trey Roby, Xiuqin Wu [ Cindy Wang, David Shupe, Robert Lupton, Trey Roby, Xiuqin Wu ]
            Hide
            shupe David Shupe added a comment -

            Questions for Robert Lupton: Should afw.display.dot check for NaNs in the ellipse shape parameters and refrain from sending them?

            When a backend receives an ellipse with NaN shape parameters, is it sufficient to silently ignore these? Or should some sort of error be communicated back to the user?

            Show
            shupe David Shupe added a comment - Questions for Robert Lupton : Should afw.display.dot check for NaNs in the ellipse shape parameters and refrain from sending them? When a backend receives an ellipse with NaN shape parameters, is it sufficient to silently ignore these? Or should some sort of error be communicated back to the user?
            Hide
            rhl Robert Lupton added a comment -

            I think you can silently ignore NaNs – they aren't the display code's problem.

             

            Show
            rhl Robert Lupton added a comment - I think you can silently ignore NaNs – they aren't the display code's problem.  
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Epic Link DM-14499 [ 84664 ]
            Hide
            cwang Cindy Wang [X] (Inactive) added a comment -

            It is a bug in Firefly. Firefly intends to ignore nani silently and render other correct ones based on current development.

            Show
            cwang Cindy Wang [X] (Inactive) added a comment - It is a bug in Firefly. Firefly intends to ignore nani silently and render other correct ones based on current development.
            Hide
            shupe David Shupe added a comment -

            The bug in Firefly has been fixed in DM-15245.

            No action is needed on the Python side. Ellipses with NaN shapes are silently discarded by the backend.

            The story points are for analyzing the problem and testing.

            Show
            shupe David Shupe added a comment - The bug in Firefly has been fixed in DM-15245 . No action is needed on the Python side. Ellipses with NaN shapes are silently discarded by the backend. The story points are for analyzing the problem and testing.
            shupe David Shupe made changes -
            Resolution Done [ 10000 ]
            Status To Do [ 10001 ] Done [ 10002 ]
            shupe David Shupe made changes -
            Sprint SUIT Sprint 2018-08 [ 738 ]
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Link This issue is triggering DM-15245 [ DM-15245 ]
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Link This issue is cloned by DM-15245 [ DM-15245 ]
            xiuqin Xiuqin Wu [X] (Inactive) made changes -
            Epic Link DM-14499 [ 84664 ] DM-8771 [ 28610 ]

              People

              Assignee:
              shupe David Shupe
              Reporter:
              shupe David Shupe
              Watchers:
              Cindy Wang [X] (Inactive), David Shupe, Robert Lupton, Trey Roby, Xiuqin Wu [X] (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.