Ticket #558 (closed defect: fixed)

Opened 8 years ago

Last modified 8 years ago

To Binary Mask causes slider slowdown

Reported by: bdezonia Owned by: leek
Priority: major Milestone: biweekly-2011: Jul-18 to Jul-29
Component: Core Version:
Severity: serious Keywords:
Cc: Blocked By:
Blocking:

Description

I don't know much about the To Binary Mask command or how it should behave. Given that caveat open Organ of Corti, then choose To Binary Mask, select 25 for the value and run it. After the plugin is done notice that adjusting C & Z sliders in the image is slow. It becomes even slower if you choose an image that has even more planes.

Change History

comment:1 in reply to: ↑ description Changed 8 years ago by leek

  • Status changed from new to accepted

The problem here is that the DefaultAdapter isn't told about the current plane. It could use the XYProjector if it were passed into updateFigure - the interface could be changed to do something like that. For now, updateOverlay can assume that all editing is done in a 2-d context.

The larger problem is that the projection is owned by the display view. I believe that the EuclideanSpace is a property of Display and the projection of that is something that is modeled by the DisplayWindow, so the projectors should be built from information in the DisplayWindow, appropriately for the DisplayView or OverlayView that uses it. But, for now, I'll have the SwingOverlayView go hunting for the DisplayView and pull out it's projector, passing it to updateFigure.

comment:2 Changed 8 years ago by leek

I like it the way it is after I fixed it, hope you all do too.

comment:3 Changed 8 years ago by leek

  • Status changed from accepted to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.