Skip to content

ENH: Use the hot colormap for PET co-registration reports - #351

Open
mnoergaard wants to merge 2 commits into
mainfrom
update_cmap_pet_viz
Open

mnoergaard wants to merge 2 commits into
mainfrom
update_cmap_pet_viz

Conversation

@mnoergaard

Copy link
Copy Markdown
Collaborator

This PR addresses issue #350, by adding support for display PET images in PET-to-anatomical co-registration reports using Matplotlib’s hot colormap while keeping anatomical images grayscale. This improves the visibility of PET uptake patterns without changing the existing hover interaction.

Tests were added to verify the colormap configuration for both the standard co-registration and reference-mask reports.

@CPernet

CPernet commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

hot is no good, plasma a bit better -- but you can always linearize luminance eg https://github.com/CPernet/brain_colours/blob/master/code/software_linearization/standard_colormaps_comparison.png

@codecov

codecov Bot commented Aug 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.49%. Comparing base (221c157) to head (ac8876a).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #351   +/-   ##
=======================================
  Coverage   87.48%   87.49%           
=======================================
  Files          89       89           
  Lines       10794    10801    +7     
  Branches     1160     1160           
=======================================
+ Hits         9443     9450    +7     
  Misses       1122     1122           
  Partials      229      229           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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