Skip to content

feat: add WhViBlGrYeOrRe colormap from NCL#23

Merged
jourdain merged 3 commits into
mainfrom
feature-add-WhViBlGrYeOrRe
Jul 1, 2026
Merged

feat: add WhViBlGrYeOrRe colormap from NCL#23
jourdain merged 3 commits into
mainfrom
feature-add-WhViBlGrYeOrRe

Conversation

@patrickoleary

@patrickoleary patrickoleary commented Jul 1, 2026

Copy link
Copy Markdown
Member

Add WhViBlGrYeOrRe (White-Violet-Blue-Green-Yellow-Orange-Red) colormap from NCAR Command Language (NCL) color table gallery.

  • Convert 101 RGB values from NCL format to ParaView JSON format
  • Add colormap entry to paraview_colormaps.json with proper metadata
  • Add to default_presets.json for inclusion in preset picker
  • Document NCL colormap source in presets/init.py

Source: https://www.ncl.ucar.edu/Document/Graphics/ColorTables/WhViBlGrYeOrRe.shtml
License: Public Domain

Addresses #22

Add WhViBlGrYeOrRe (White-Violet-Blue-Green-Yellow-Orange-Red) colormap
from NCAR Command Language (NCL) color table gallery.

- Convert 101 RGB values from NCL format to ParaView JSON format
- Add colormap entry to paraview_colormaps.json with proper metadata
- Add to default_presets.json for inclusion in preset picker
- Document NCL colormap source in presets/__init__.py

Source: https://www.ncl.ucar.edu/Document/Graphics/ColorTables/WhViBlGrYeOrRe.shtml
License: Public Domain
Add ref: main and persist-credentials to checkout step in release job
to fix 'Detached HEAD state cannot match any release groups' error
from python-semantic-release.
Comment thread .github/workflows/ci.yml Outdated
with:
fetch-depth: 0
persist-credentials: true
ref: main

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you remove those 2 lines?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • persist-credentials is true by default
  • ref force to only test the main branch rather than the code you just changed.

@jourdain jourdain self-requested a review July 1, 2026 16:24
@jourdain jourdain merged commit 139c718 into main Jul 1, 2026
5 of 6 checks passed
@jourdain jourdain deleted the feature-add-WhViBlGrYeOrRe branch July 1, 2026 16:28
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.

3 participants