Skip to content

DOC: remove incorrect colorbar reference from contours_topomap docstring#13682

Open
deveshbervar wants to merge 2 commits intomne-tools:mainfrom
deveshbervar:fix-plot-topomap-docstring
Open

DOC: remove incorrect colorbar reference from contours_topomap docstring#13682
deveshbervar wants to merge 2 commits intomne-tools:mainfrom
deveshbervar:fix-plot-topomap-docstring

Conversation

@deveshbervar
Copy link

Fixes #12698

What was changed

Removed the incorrect reference to the non-existent colorbar argument
from the contours parameter description in mne.viz.plot_topomap().

Why this change

The docstring mentioned colorbar=True, but plot_topomap() does not
have a colorbar argument. This PR updates the documentation to remove
the incorrect reference.

Implementation

  • Updated docdict["contours_topomap"] in mne/utils/docs.py
  • Removed the sentence referring to colorbar=True

@deveshbervar
Copy link
Author

Hi @cbrnr, I noticed the CI checks are blocked due to the block-unregistered-user restriction. Could you please approve the workflow so the checks can run? Thanks!

@cbrnr
Copy link
Contributor

cbrnr commented Feb 28, 2026

I don't know why I can't re-run the CircleCI jobs. @larsoner can you help? I'm logged in with my account but the options are still grayed out. Do you need to add me to an org?

@tsbinns
Copy link
Contributor

tsbinns commented Feb 28, 2026

@deveshbervar If you link your GitHub account to CircleCI and push a commit (even an empty one), the docs build will run without us needing to manually approve it.

The values should be in µV for EEG, fT for magnetometers and fT/m for
gradiometers. If ``colorbar=True``, the colorbar will have ticks
corresponding to the contour levels. Default is ``6``.
gradiometers.Default is ``6``.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
gradiometers.Default is ``6``.
gradiometers. Default is ``6``.

Should have a space.

@deveshbervar
Copy link
Author

Thanks! I’ve applied the suggested formatting fix.

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.

Docstring of mne.viz.plot_topomap() incorrect

3 participants