NOTICE! This is a static HTML version of a legacy ImageJ Trac ticket.

The ImageJ project now uses GitHub Issues for issue tracking.

Please file all new issues there.

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: imagej2-b3-headless
Component: Plugins Version:
Severity: serious Keywords:
Cc: Blocked By:

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:1 Changed 2012-04-17T15:34:54-05:00 by bdezonia

  • Description modified

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.