New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change default colormap for model scalars and displacement volumes #989

Closed
wants to merge 3 commits into
base: master
from

Conversation

Projects
None yet
3 participants
@fepegar
Copy link

fepegar commented Jul 11, 2018

See #741

Also, in my vessels network with Radius set as Active scalar:

Using Viridis, I can easily figure out that dark lines represent thin vessels and yellow ones are thicker:
viridis

Using Rainbow, the current default, the result is more fun but more confusing:
rainbow

@fepegar

This comment has been minimized.

Copy link
Author

fepegar commented Jul 11, 2018

Feature request: is it possible to visualize the colorbar of the scalars in the 3D view, as the one shown by Data Probe on the 2D views?

@lassoan

This comment has been minimized.

Copy link
Contributor

lassoan commented Jul 11, 2018

D You can show the color bar in 3D views using Colors module (there is a section to configure it, near the bottom).

@fepegar

This comment has been minimized.

Copy link
Author

fepegar commented Jul 11, 2018

Yes, but it would be great to see the values changing to reflect the "Display Range" widget limits.

@pieper

This comment has been minimized.

Copy link
Member

pieper commented Jul 11, 2018

The pull request makes sense to me. +1 for merging.

@lassoan

This comment has been minimized.

Copy link
Contributor

lassoan commented Jul 11, 2018

There are 4 distinct colors (red, green, blue, purple) in rainbow colormap, which might be helpful, as it provides iso-lines/surfaces. However, the iso-values are usually just randomly assigned (by scaling the colormap to the scalar range), highlighting lines/surfaces that are not particularly interesting.

Viridis provides continuous transition, so there are no specific iso-values that are displayed more prominently.

Due to this, viridis seems to be a more generic colormap, which I agree that seems more suitable as a default colormap than rainbow.

I would slightly prefer plasma (another smoothly transitioning colormap), as its colors are associated with cold-to-hot temperature; while viridis transitions between two neutral colors (purple-to-yellow).

'R' programming language went with viridis (https://www.r-bloggers.com/using-the-new-viridis-colormap-in-r-thanks-to-simon-garnier/), so it might be one more reason to choose the same (or not, to be more distinctive?).

Rainbow:
image

Viridis:
image

Plasma:
image

@fepegar

This comment has been minimized.

Copy link
Author

fepegar commented Jul 11, 2018

Matplotlib also uses Viridis as default. Originally they had only the other three, but apparently people prefer Viridis for some reason. They have all been designed on the same principles anyway.

I agree that Plasma might be a bit closer to the hot-cold idea but not because of the extremes, which are very similar, but because it goes through red/orange instead of green.

Do you think it'd be a good idea to add this default to the PET and DTI presets?

@pieper

This comment has been minimized.

Copy link
Member

pieper commented Jul 11, 2018

@fedorov do you have preferences for PET? @ljod and @ihnorton how about dMRI?

slicerbot pushed a commit that referenced this pull request Nov 14, 2018

lassoan
ENH: Change default colormap for model scalars and transforms
Viridis provides continuous transition, so there are no specific iso-values that are displayed more prominently.
Due to this, viridis seems to be a more generic colormap, which seems more suitable as a default colormap than rainbow.

See discussion here: #989

Co-Authored-By: Fernando Perez-Garcia <fepegar@gmail.com>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@27549 3bd1e089-480b-0410-8dfb-8563597acbee
@lassoan

This comment has been minimized.

Copy link
Contributor

lassoan commented Nov 14, 2018

Merged 589e6b8

@lassoan lassoan closed this Nov 14, 2018

jcfr added a commit that referenced this pull request Jan 16, 2019

ENH: Change default colormap for model scalars and transforms
Viridis provides continuous transition, so there are no specific iso-values that are displayed more prominently.
Due to this, viridis seems to be a more generic colormap, which seems more suitable as a default colormap than rainbow.

See discussion here: #989

Co-Authored-By: Fernando Perez-Garcia <fepegar@gmail.com>

git-svn-id: http://svn.slicer.org/Slicer4/branches/Slicer-4-10@27762 3bd1e089-480b-0410-8dfb-8563597acbee
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment