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

Mask overlays missing with more than one call to mtv on the same display object

    XMLWordPrintable

    Details

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

      Description

      When using the Firefly backend for lsst.afw.display, the first MaskedImage or Exposure displayed via Display.mtv has a wonderful mask overlay (much better than DS9's!).

      Unfortunately, if I call mtv a second time on the same Display instance, the mask overlays do not appear, even though the image itself is updated.

      If I close the image window in Firefly itself before calling Display.mtv, things work as expected.

      If a quick fix is possible, it'd be great to get it in this week so I can avoid demoing the workaround at LSST@Europe3 next week (but it's not a terrible workaround if quick fix is not possible).

        Attachments

          Issue Links

            Activity

            Hide
            shupe David Shupe added a comment -

            The behavior of the masks is server-dependent, in my testing. Running the intro-with-globuler notebook and using the lsp-demo.lsst.codes Firefly server, the masks don't appear after the first Display.mtv. On the last-demo.ncsa.illinois.edu server, the masks still display in my testing.

            We are testing a fix on the display_firefly side, that would delete the displayed image at the start of mtv

            Show
            shupe David Shupe added a comment - The behavior of the masks is server-dependent, in my testing. Running the intro-with-globuler notebook and using the lsp-demo.lsst.codes Firefly server, the masks don't appear after the first Display.mtv . On the last-demo.ncsa.illinois.edu server, the masks still display in my testing. We are testing a fix on the display_firefly side, that would delete the displayed image at the start of mtv . 
            Hide
            shupe David Shupe added a comment -

            It was decided to make a fix on the Firefly server instead of deleting the image before mtv. Unfortunately I am finding the problem is still occurring when using the https://lsp-demo.lsst.codes/firefly.

            Show
            shupe David Shupe added a comment - It was decided to make a fix on the Firefly server instead of deleting the image before mtv . Unfortunately I am finding the problem is still occurring when using the https://lsp-demo.lsst.codes/firefly .
            Hide
            shupe David Shupe added a comment -

            We were able to reproduce the problem on my laptop. The issue was fixed in review, by this commit. Masks appear reliably now, on my laptop and on the lsp-demo platform.

            Show
            shupe David Shupe added a comment - We were able to reproduce the problem on my laptop. The issue was fixed in review, by this commit . Masks appear reliably now, on my laptop and on the lsp-demo platform.

              People

              Assignee:
              roby Trey Roby
              Reporter:
              jbosch Jim Bosch
              Reviewers:
              Cindy Wang [X] (Inactive)
              Watchers:
              Cindy Wang [X] (Inactive), David Shupe, Gregory Dubois-Felsmann, Jim Bosch, Trey Roby, Xiuqin Wu [X] (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.