Skip to content

Add turbo colormap to local ColorMaps and GradientEditorItem#2778

Merged
j9ac9k merged 2 commits intopyqtgraph:masterfrom
ptuemmler:turbo_in_GradientEditorItem
Aug 24, 2023
Merged

Add turbo colormap to local ColorMaps and GradientEditorItem#2778
j9ac9k merged 2 commits intopyqtgraph:masterfrom
ptuemmler:turbo_in_GradientEditorItem

Conversation

@ptuemmler
Copy link
Copy Markdown
Contributor

Added turbo from https://ai.googleblog.com/2019/08/turbo-improved-rainbow-colormap-for.html to default Gradients dict in GradientEditorItem.py such that it can be used directly in ImageViews and their HistogramLUTItems.

Rainbow colormaps yield far greater contrast and depth differentiation than colormaps such as magma or viridis. Turbo is far more pleasant than the already available spectrum map and does not have the same banding issues as jet.

@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented Jul 21, 2023

Would you consider adding the colormap in csv form here? https://github.com/pyqtgraph/pyqtgraph/tree/71bcf3f6d078b07a19b90791d96d27d28cb545d1/pyqtgraph/colors/maps

GradientEditorItem doesn't get as much usage as our ColorMap functionality.

@ptuemmler
Copy link
Copy Markdown
Contributor Author

Sure thing!

@ptuemmler ptuemmler changed the title Added turbo colormap to default Gradients of GradientEditorItem Added turbo colormap to local ColorMaps and GradientEditorItem Jul 22, 2023
@ptuemmler ptuemmler changed the title Added turbo colormap to local ColorMaps and GradientEditorItem Add turbo colormap to local ColorMaps and GradientEditorItem Jul 22, 2023
@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented Aug 24, 2023

@ptuemmler My apologies for taking so long to merge, I normally try and wrap things up far quicker than I have now. I've tested this works as intended, LGTM, merging.

@j9ac9k j9ac9k merged commit 10c7607 into pyqtgraph:master Aug 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants