Skip to content

DOC: fix missing space, use f-strings, structure->object #13291

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
Merged
sappelhoff merged 2 commits into from
Jun 25, 2025
Merged

DOC: fix missing space, use f-strings, structure->object #13291

sappelhoff merged 2 commits into from
Jun 25, 2025
+6 −3

Conversation

@sappelhoff
Copy link
Member

@sappelhoff sappelhoff commented Jun 24, 2025

Before:

ValueError: Number of channels in the Info object (60) and the data array (59) do not match.Pick Info with e.g. mne.pick_info and mne.channel_indices_by_type.

After:

ValueError: Number of channels in the Info object (60) and the data array (59) do not match. Pick the Info object (e.g., using mne.pick_info and mne.channel_indices_by_type).

larsoner
larsoner approved these changes Jun 24, 2025
View reviewed changes
Copy link
Member

@larsoner larsoner left a comment

Choose a reason for hiding this comment

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

Marking for merge-when-green (which should happen after #13261 lands and I merge main into this branch), thanks in advance @sappelhoff !

@sappelhoff sappelhoff merged commit 729ce25 into mne-tools:main Jun 25, 2025
32 checks passed
@sappelhoff sappelhoff deleted the typo branch June 25, 2025 08:06
larsoner added a commit to larsoner/mne-python that referenced this pull request Jun 26, 2025
@larsoner
Merge remote-tracking branch 'upstream/main' into feature/TD-nirs_snirf
63b9f35 
* upstream/main: (55 commits)
  doc: fix rendering typo rst docstr (mne-tools#13301)
  DOC: fix docstrs around layout functions (mne-tools#13300)
  MAINT: Fix doc build failure due to deprecation (mne-tools#13299)
  Birthday input cast to datetime.date (mne-tools#13284)
  DOC: fix missing space, use f-strings, structure->object (mne-tools#13291)
  [pre-commit.ci] pre-commit autoupdate (mne-tools#13290)
  ENH: channel_indices_by_type now has an exclude param (mne-tools#13293)
  Proj id and proj name access (mne-tools#13261)
  Fix: nearly invisible traces with spatial_colors=True (mne-tools#13286)
  [pre-commit.ci] pre-commit autoupdate (mne-tools#13283)
  Bump autofix-ci/action from 551dded8c6cc8a1054039c8bc0b8b48c51dfc6ef to 635ffb0c9798bd160680f18fd73371e355b85f27 in the actions group (mne-tools#13282)
  fix Maxwell bads filtering (mne-tools#13280)
  fix actionable linkcheck errors (mne-tools#13273)
  MAINT: Use radius keyword with PyVista tube (mne-tools#13277)
  BUG: Fix bug with simulating head pos and BEM (mne-tools#13276)
  [pre-commit.ci] pre-commit autoupdate (mne-tools#13274)
  MAINT: Update code credit (mne-tools#13267)
  Annotations extras (mne-tools#13228)
  Tidy up the directory reading (mne-tools#13268)
  FIX, DOC: Drop bad channel in 10_publication_figure.py (mne-tools#13266)
  ...
WouterKroot pushed a commit to WouterKroot/mne-python that referenced this pull request Aug 13, 2025
@sappelhoff @larsoner
DOC: fix missing space, use f-strings, structure->object (mne-tools#1…
5a145ef 
…3291)

Co-authored-by: Eric Larson <larson.eric.d@gmail.com>
zEdS15B3GCwq pushed a commit to zEdS15B3GCwq/mne-python that referenced this pull request Aug 25, 2025
@sappelhoff @larsoner @zEdS15B3GCwq
DOC: fix missing space, use f-strings, structure->object (mne-tools#1…
fbc7834 
…3291)

Co-authored-by: Eric Larson <larson.eric.d@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@larsoner larsoner larsoner approved these changes

@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

no-changelog-entry-needed skip-changelog-checks

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sappelhoff @larsoner