Ticket #1137 (closed defect: fixed)
Opened 2012-04-24T09:22:14-05:00
Last modified 2012-08-01T11:23:09-05:00
Clear/Clear Outside misbehaving
Reported by: | bdezonia | Owned by: | bdezonia |
---|---|---|---|
Priority: | major | Milestone: |
|
Component: | Legacy Compatibility | Version: | |
Severity: | serious | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | #1345 |
Description
open clown
draw overlay
run edit Clear
pixels changed inside roi
(drawn in last color set in IJ2 rather than curr fg/bg values color)
(see OptionsChannels to see that two different values exist)
run edit Clear Outside\
interior roi pixels revert
exterior drawn in last color again
exception thrown:
[ERROR] ImageJ 1.x plugin threw exception
java.lang.IndexOutOfBoundsException: Index: 0, Size: 3
at java.util.ArrayList.rangeCheck(ArrayList.java:571)
at java.util.ArrayList.get(ArrayList.java:349)
at imagej.data.display.AbstractDatasetView.updateLUTs(AbstractDatasetView.java:324)
at imagej.data.display.AbstractDatasetView.rebuild(AbstractDatasetView.java:225)
at imagej.data.display.AbstractImageDisplay.rebuild(AbstractImageDisplay.java:137)
at imagej.ext.display.AbstractDisplay.update(AbstractDisplay.java:113)
at imagej.data.display.AbstractImageDisplay.update(AbstractImageDisplay.java:233)
at imagej.data.display.DefaultOverlayService.removeOverlay(DefaultOverlayService.java:160)
at imagej.legacy.translate.OverlayHarmonizer.updateDisplay(OverlayHarmonizer.java:140)
at imagej.legacy.translate.Harmonizer.updateDisplay(Harmonizer.java:178)
at imagej.legacy.plugin.LegacyPlugin.updateDisplaysFromImagePluses(LegacyPlugin.java:307)
at imagej.legacy.plugin.LegacyPlugin.run(LegacyPlugin.java:161)
at imagej.ext.plugin.PluginModule.run(PluginModule.java:151)
at imagej.ext.module.ModuleRunner.run(ModuleRunner.java:149)
at imagej.ext.module.ModuleRunner.call(ModuleRunner.java:119)
at imagej.ext.module.ModuleRunner.call(ModuleRunner.java:1)
at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:334)
at java.util.concurrent.FutureTask.run(FutureTask.java:166)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1110)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:603)
at java.lang.Thread.run(Thread.java:679)
Change History
comment:1 Changed 2012-04-24T10:49:48-05:00 by bdezonia
- Owner changed from bdezonia to curtis
- Status changed from new to assigned
- Milestone changed from imagej-2.0.0-beta2 to imagej-2.0.0-beta3
comment:2 Changed 2012-07-12T17:01:14-05:00 by curtis
- Milestone changed from imagej-2.0.0-beta3 to imagej-2.0.0-beta4
comment:4 Changed 2012-07-26T13:43:42-05:00 by bdezonia
Testing post beta 3: The exception error is gone.
Remaining issue: Open clown and draw rect. Run Clear and interior is filled with background (correct). Follow it with Clear Outside. The outer region is filled with background (correct). The interior region is restored to a view of the original clown data (incorrect). In IJ1 the central region stays as the background color from the previous clear.
comment:5 Changed 2012-07-26T15:43:44-05:00 by bdezonia
- Status changed from assigned to closed
- Resolution set to fixed
Fixed pixel munging error with commit 1e54c82e0cb643dcd1230aec3f24603ba6a66e36
comment:6 Changed 2012-08-01T11:22:55-05:00 by bdezonia
- Status changed from closed to reopened
- Resolution fixed deleted
comment:7 Changed 2012-08-01T11:23:03-05:00 by bdezonia
- Owner changed from curtis to bdezonia
- Status changed from reopened to assigned
comment:8 Changed 2012-08-01T11:23:09-05:00 by bdezonia
- Status changed from assigned to closed
- Resolution set to fixed
Lets not worry about the fill value in this ticket. I'll address that elsewhere.
You can get a number of exceptions from this sequence. Clear Outside or Clear work by themselves fine. Its when you do one after the other that exceptions pop up. The following is common. Note its in code recently modified by Tobias and worked on by Grant and Curtis in the past,
[ERROR] Exception during event handling:
java.lang.IllegalArgumentException: Expected 3 converters but got 6