Ticket #1125 (closed defect: fixed)
Opened 2012-04-17T15:11:06-05:00
Last modified 2012-05-25T15:19:54-05:00
Duplicate command does not respect selection
| Reported by: | bdezonia | Owned by: | bdezonia |
|---|---|---|---|
| Priority: | major | Milestone: |
|
| Component: | Plugins | Version: | |
| Severity: | serious | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: |
Description (last modified by bdezonia)
After the beta1 release in the imagej list Leon Espinosa reported:
"Duplicate" command duplicates the whole image not only the selection.
Change History
comment:2 Changed 2012-04-24T12:22:49-05:00 by bdezonia
In 1084c35c388a3aee47f44c4de3d37fe52e81fde8 now only duplicating selected bounds.
In IJ1 the ROI from the original image is duplicated in the new image. That code is not yet in place in IJ2.
comment:3 Changed 2012-04-26T16:14:19-05:00 by curtis
- Milestone changed from imagej-2.0.0-beta2 to imagej-2.0.0-beta3
comment:4 Changed 2012-05-22T12:20:15-05:00 by bdezonia
We need to determine how we want to handle the duplication of ROIs. When user runs duplicate on a selected region the existing ROIs need to be duplicated and moved to new origin. I have done some play implementations. They all eventually reach the limitations of the Imglib RegionOfInterest definitions. One needs to define duplicate() and move() in the base classes. Note that move() will be broken for the Img-based BinaryMaskedRegionOfInterest. Using PointSets would simplify things here. Need further discussion with Imglib folks.
comment:5 Changed 2012-05-25T15:19:54-05:00 by bdezonia
- Status changed from new to closed
- Resolution set to fixed
With commit 192365bdbc3b77c02b348bfb1be6e8c18e03705d an initial implementation of code that allows Overlays to be moved and duplicated is in place. It could be further frefined but we'll wait for discussions with others. Closing this ticket.