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

Implement minor afw.display changes as per RFC-241

    Details

    • Type: Story
    • Status: Done
    • Priority: Major
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: None
    • Templates:
    • Story Points:
      1
    • Epic Link:
    • Sprint:
      DRP F16-4, DRP F16-5
    • Team:
      Data Release Production

      Description

      Please implement the changes proposed in the RFC

        Issue Links

          Activity

          Hide
          jbosch Jim Bosch added a comment -

          The changes on this ticket have made it necessary to guard calls to lsst.afw.display.ds9.Buffering() with (e.g.) if display; that seems to have previously been a harmless no-op when no display is active. I'm fixing all code that called it unguarded on DM-8002 (I think that's a good idea in all of the cases I've found regardless), but if it is an undesirable change it should either be fixed here or a new ticket opened for it.

          Show
          jbosch Jim Bosch added a comment - The changes on this ticket have made it necessary to guard calls to lsst.afw.display.ds9.Buffering() with (e.g.) if display ; that seems to have previously been a harmless no-op when no display is active. I'm fixing all code that called it unguarded on DM-8002 (I think that's a good idea in all of the cases I've found regardless), but if it is an undesirable change it should either be fixed here or a new ticket opened for it.
          Hide
          swinbank John Swinbank added a comment -

          Robert Lupton — I believe this work has hit master (albeit without a merge commit). Can you confirm &, if appropriate, close the ticket? Thanks.

          Show
          swinbank John Swinbank added a comment - Robert Lupton — I believe this work has hit master (albeit without a merge commit). Can you confirm &, if appropriate, close the ticket? Thanks.
          Hide
          rhl Robert Lupton added a comment -

          See note on merge-woes in DM-7869

          Show
          rhl Robert Lupton added a comment - See note on merge-woes in DM-7869
          Hide
          wmwood-vasey Michael Wood-Vasey added a comment -

          @rhl, apologies for never reviewing this ticket.

          Show
          wmwood-vasey Michael Wood-Vasey added a comment - @rhl, apologies for never reviewing this ticket.

            People

            • Assignee:
              rhl Robert Lupton
              Reporter:
              rhl Robert Lupton
              Reviewers:
              Michael Wood-Vasey
              Watchers:
              Jim Bosch, John Swinbank, Michael Wood-Vasey, Robert Lupton
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development

                  Agile