Skip to content

Refactor OPM/Neuromag grouping#13921

Open
PragnyaKhandelwal wants to merge 8 commits into
mne-tools:mainfrom
PragnyaKhandelwal:refactor-opm-neuromag-grouping-clean
Open

Refactor OPM/Neuromag grouping#13921
PragnyaKhandelwal wants to merge 8 commits into
mne-tools:mainfrom
PragnyaKhandelwal:refactor-opm-neuromag-grouping-clean

Conversation

@PragnyaKhandelwal
Copy link
Copy Markdown
Contributor

Reference issue (if any)

Follow‑up to #13866.

What does this implement/fix?

  • Makes tangential OPM topomaps use unsigned RMS magnitude (aligned with Neuromag grads).
  • Uses per‑group vlim and non‑diverging colormap for tangential maps.
  • Adds a shared helper to unify grouping/merging logic for OPM and Neuromag‑style channels.
  • Updates OPM grouped topomap tests for new axes/colorbar behavior.

Additional information

@larsoner
Copy link
Copy Markdown
Member

Failures look related. The ubuntu-latest / pip job runs the most complete set of tests, so you can look here for a good list of failures:

https://github.com/mne-tools/mne-python/actions/runs/26439596935/job/77830628215?pr=13921

Once you have these working locally feel free to push fixes!

@PragnyaKhandelwal PragnyaKhandelwal marked this pull request as ready for review May 27, 2026 17:36
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