Ticket #751 (closed task: fixed)

Opened 8 years ago

Last modified 6 years ago

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 8 years ago by bdezonia

Can verify by code inspection that case one is already supported.

comment:2 Changed 8 years ago 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:3 Changed 7 years ago by curtis

  • Status changed from assigned to accepted

comment:4 Changed 7 years ago by curtis

  • Blocking 1011 added

comment:5 Changed 7 years ago by curtis

  • Milestone changed from imagej-2.0-beta1 to imagej-2.0-beta3

comment:6 Changed 7 years ago by curtis

  • Milestone changed from imagej-2.0.0-beta4 to imagej-2.0.0-beta5

comment:7 Changed 6 years ago 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 6 years ago by bdezonia

  • Status changed from reviewing to closed
  • Resolution set to fixed

Yes, this is working. Closing.

Note: See TracTickets for help on using tickets.