Ticket #698 (closed defect: fixed)

Opened 8 years ago

Last modified 7 years ago

Incorporate interface ColorTable<T>

Reported by: gharris Owned by: aivar
Priority: minor Milestone: imagej2-b4-plugins
Component: Display API Version:
Severity: minor Keywords:
Cc: Blocked By:
Blocking: #1385

Description

Incorporate interface ColorTable<T> as appropriate where ColorTable8 is used.
E.g. in DatasetView, AbstractDatasetView, RealLUTConverter...

Change History

comment:1 Changed 8 years ago by gharris

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

comment:2 Changed 8 years ago by curtis

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

comment:3 Changed 8 years ago by curtis

  • Owner changed from gharris to curtis

comment:4 Changed 7 years ago by curtis

  • Blocking 1323 added

comment:5 Changed 7 years ago by aivar

  • Owner changed from curtis to aivar
  • Status changed from new to assigned

I'm taking over this ticket as I'm doing some work on LUTs.

comment:6 Changed 7 years ago by aivar

My approach is to rename ColorTable<T> to be AbstractColorTable<T>, then reuse ColorTable as an interface that deals with ColorTable8, ColorTable16, or whatever.

comment:7 Changed 7 years ago by aivar

The ColorTable interface doesn't use generics. The type T refers to the internal representation of the colors. A user of ColorTables shouldn't have to deal with that.

comment:8 Changed 7 years ago by curtis

  • Blocking 1385 added; 1323 removed

This ticket fits better under the Improve lookup tables (LUTs) feature.

comment:9 Changed 7 years ago by curtis

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

This work has been completed with the merge of the luts branch.

Note: See TracTickets for help on using tickets.