Ticket #751 (closed task: fixed)
Opened 2011-09-14T11:20:33-05:00
Last modified 2013-06-10T14:44:44-05:00
Legacy translation test cases
| Reported by: | bdezonia | Owned by: | bdezonia |
|---|---|---|---|
| Priority: | major | Milestone: | imagej2-b8-analysis |
| Component: | Legacy Compatibility | Version: | |
| Severity: | serious | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | #1011 |
Description
Here are a couple use cases to support:
User opens bridge, adds a channel and two color tables and composite channel count == 2. IJ1 translations should be a CompositeImage.
Color tables going to IJ1. If there is a single ColorTable for a Dataset then translate as an ImagePlus. If there are multiple color tables then grab the first color table of each channel and make a composite image. Unless channels > 7 and then again an ImagePlus.
Change History
comment:1 Changed 2011-10-11T15:40:47-05:00 by bdezonia
comment:2 Changed 2011-10-11T16:14:24-05:00 by bdezonia
- Owner changed from bdezonia to curtis
- Status changed from new to assigned
Via code inspection can see that case 2 is supported. But the difference is that the color tables used from IJ2 to IJ1 are those owned by the active view of a given display. It sounds like Curtis might have wanted the first lut of each channel from the Dataset. Curtis can you clarify here?
comment:5 Changed 2012-03-05T10:59:17-06:00 by curtis
- Milestone changed from imagej-2.0-beta1 to imagej-2.0-beta3
comment:6 Changed 2012-09-10T15:48:11-05:00 by curtis
- Milestone changed from imagej-2.0.0-beta4 to imagej-2.0.0-beta5
comment:7 Changed 2013-06-07T16:06:08-05:00 by curtis
- Owner changed from curtis to bdezonia
- Status changed from accepted to reviewing
- Milestone changed from imagej2-b7-ndim-data to imagej2-b8-analysis
20 months later, I offer my "clarification": the LUTs associated with Datasets are those associated with the data itself, intrinsically. They are useful hints for the default DatasetView LUTs, but no substitute for the actual current DatasetView LUTs. So we should definitely be using the DatasetView LUTs when synchronizing.
Is this all working now? Can this ticket be closed?
comment:8 Changed 2013-06-10T14:44:44-05:00 by bdezonia
- Status changed from reviewing to closed
- Resolution set to fixed
Yes, this is working. Closing.
Can verify by code inspection that case one is already supported.