Description
DefaultLUTService.legacyTextLUT uses TableLoader.valuesFromTextFile to read a LUT from a CSV files. Since this is the only use of TableLoader.valuesFromTextFile in our codebase we should retire it in favor of implementations in scijava-plugins-io-table. See also scijava/scijava-plugins-io-table#4 (comment).
This, however, poses several challenges/tasks (in no particular order):
- Should
LUTServicestill support reading of LUTs fromInputStream? If so, such requests can't be handled byIOService. Is that correct, @ctrueden? - We should migrate code for reading LUTs from
InputStreams into dedicatedIOPlugins - Basically, strip down the
LUTServiceto pass requests toIOServiceand fail gracefully if a LUT can't be read (which doesn't work forInputStreams - see 1.)