Skip to content

MAINT: Stabilize MxNE tests#13321

Merged
larsoner merged 1 commit intomne-tools:mainfrom
larsoner:tests
Jul 15, 2025
Merged

MAINT: Stabilize MxNE tests#13321
larsoner merged 1 commit intomne-tools:mainfrom
larsoner:tests

Conversation

@larsoner
Copy link
Copy Markdown
Member

@Genuster from some testing on SSH I realized it was a Heisenbug -- only showed up on some calls to pytest mne/inverse_sparse -xk sure_meg for example. I am hoping adding these random_state=0s to the simulate_evoked calls fixes the bug. I expect them to. Will merge this either way then merge main into your branch, and we can continue testing there if needed. But hopefully it just comes back green!

@larsoner
Copy link
Copy Markdown
Member Author

... and this sort of thing can happen when new tests are added, if those tests (or any underlying functions) make use of the global-state-based np.random etc. Hence why I'm pushing commits to #13259 to change some np.random.<func> calls to be np.random.RandomState(0).<meth> calls instead.

@larsoner larsoner mentioned this pull request Jul 15, 2025
16 tasks
@larsoner larsoner merged commit 48f1588 into mne-tools:main Jul 15, 2025
33 of 34 checks passed
@larsoner larsoner deleted the tests branch July 15, 2025 16:32
larsoner added a commit to larsoner/mne-python that referenced this pull request Jul 23, 2025
* upstream: (115 commits)
  Capitalize Get Help (mne-tools#13340)
  Website page load fixes (mostly) (mne-tools#13343)
  MAINT: Restore statsmodels to pip-pre (mne-tools#13345)
  BUG: Fix bug with reading old reports (mne-tools#13341)
  [pre-commit.ci] pre-commit autoupdate (mne-tools#13338)
  MAINT: Fix ref cycle with vtkPolyData (mne-tools#13336)
  MAINT: Dont set private attributes for PyVista Actor (mne-tools#13334)
  Add icon links to our donation pages (mne-tools#13331)
  MAINT: Dont set attribute on PolyData (mne-tools#13330)
  MAINT: Bump mins, deprecations (mne-tools#13322)
  Fix changes in SSD for backward compatibility [circle deploy] (mne-tools#13327)
  ENH: Add GED transformer (mne-tools#13259)
  FIX: Links
  MAINT: Stabilize MxNE tests (mne-tools#13321)
  FIX: DigMontage.rename_channels should return self (mne-tools#13320)
  MAINT: Prep for release (mne-tools#13319)
  [pre-commit.ci] pre-commit autoupdate (mne-tools#13317)
  MAINT: Update code credit (mne-tools#13318)
  MAINT: Make MxNE test more robust (mne-tools#13315)
  Adjust Pupil channel units (again) (mne-tools#13314)
  ...
WouterKroot pushed a commit to WouterKroot/mne-python that referenced this pull request Aug 13, 2025
zEdS15B3GCwq pushed a commit to zEdS15B3GCwq/mne-python that referenced this pull request Aug 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants