Viewers: Support initialization of segment color from RecommendedDisplayCIELabValue

[fedorov]

Request

What feature or change would you like to see made?

Per @JamesAPetts, "SEGs just load in the default cornerstone color LUT per segment index at the current time", even if color is included in the segment sequence. It would be better if the color was initialized based on what is in SEG.

Why should we prioritize this feature?

• in the situations where there are multiple segmentations for a given structure (e.g., multiple readers or algorithms), showing them all in the same color makes interpretation difficult
• make visualization consistent with the other tools that support SEG (e.g., Slicer initializes the color)

[pieper]

For reference:

https://github.com/dcmjs-org/dcmjs/blob/master/examples/vtkDisplay/index.html#L485-L496

[fedorov]

@swederik @JamesAPetts considering the "In progress" stack is almost empty, and I recall James mentioned this should be quite straightforward, I suggest we move this one to "priority" and "in progress" - do you have any concerns?

[swederik]

Nope! Sounds good to me

[fedorov]

Thanks! I use the LIDC dataset for testing often, and it is very annoying to have all segments painted the same color, since there is only one segment per object, but multiple segments for the individual lesion. Navigation of this kind of data becomes a lot more intuitive when colors are initialized from the data.

[JamesAPetts]

Igor is working on a couple of issues, I'll take this one.

[JamesAPetts]

Resolved in #1709. Let me know if there are any issues.