Ticket #558 (closed defect: fixed)
Opened 2011-06-02T16:16:55-05:00
Last modified 2011-07-26T10:25:23-05:00
To Binary Mask causes slider slowdown
| Reported by: | bdezonia | Owned by: | leek |
|---|---|---|---|
| Priority: | major | Milestone: |
|
| 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 2011-06-03T10:48:36-05:00 by leek
- Status changed from new to accepted
comment:2 Changed 2011-06-09T12:59:23-05:00 by leek
I like it the way it is after I fixed it, hope you all do too.
comment:3 Changed 2011-07-26T10:25:23-05:00 by leek
- Status changed from accepted to closed
- Resolution set to fixed
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.