Skip to content

ENH: Add hpi_colors and hpi_labels for clear visualization #13533

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.

Sign up for GitHub

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

Jump to bottom
Open
drona-gyawali wants to merge 1 commit into
base: main
Choose a base branch
from
Open

ENH: Add hpi_colors and hpi_labels for clear visualization #13533

drona-gyawali wants to merge 1 commit into from
+129 −25

Conversation

@drona-gyawali
Copy link

@drona-gyawali drona-gyawali commented Dec 7, 2025 •
edited
Loading

Reference issue (if any)

Fixes #13485.

What does this implement/fix?

  • Automatically uses official MEGIN cable colors (1=red, 2=blue, 3=green, 4=yellow, 5=magenta, 6=cyan)
  • Optional hpi_labels=True shows coil ident as 3D text
  • Fully user-configurable (list/dict/single color)
  • Added test coverage

@welcome
Copy link

welcome bot commented Dec 7, 2025

Hello! 👋 Thanks for opening your first pull request here! ❤️ We will try to get back to you soon. 🚴

@drona-gyawali drona-gyawali marked this pull request as ready for review December 7, 2025 06:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@drammock drammock Awaiting requested review from drammock drammock is a code owner

@dengemann dengemann Awaiting requested review from dengemann dengemann is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

ENH: Clarify HPI coil order in plot_alignment

1 participant

@drona-gyawali