Skip to content

Conversation

@cbrnr
Copy link
Contributor

@cbrnr cbrnr commented Mar 20, 2023

Fixes #11576. I'm replacing - with _ at in the import call, and for a better look I think we should use - in the output. I've also removed the colons after the package names (which @hoechenberger suggested previously but we forgot) and fixed the alignment.

@agramfort
Copy link
Member

thx @cbrnr

can we test this?

@cbrnr
Copy link
Contributor Author

cbrnr commented Mar 20, 2023

can we test this?

Sure. Should I add a requires_qt_browser decorator (or whatever we want to name it)? We have some workflows that do include mne-qt-browser, so this should get tested at least once.

@agramfort
Copy link
Member

agramfort commented Mar 20, 2023 via email

@larsoner larsoner merged commit 367ed0a into mne-tools:main Mar 20, 2023
@larsoner
Copy link
Member

Thanks for getting in @hoechenberger's :-removal, too!

larsoner added a commit to larsoner/mne-python that referenced this pull request Mar 20, 2023
* upstream/main:
  Fix bug in new `mne.sys_info()` (mne-tools#11577)
  Remove legacy plot_psd* funcs/methods in our docs and codebase (mne-tools#11563)
  The order of raw1.info["bads"] should not matter when concatenating with raw0.info["bads"] mne-tools#11501 (mne-tools#11502)
  MAINT: Fixes for matplotlib and pandas (mne-tools#11574)
  Fix Circle [circle deploy]
  Fix export to EDF format with a set physical range smaller than the data range (mne-tools#11569)
  Slightly rework `mne.sys_info()` (mne-tools#11568)
@cbrnr cbrnr deleted the fix-sysinfo branch March 21, 2023 08:02
@hoechenberger
Copy link
Member

Thanks!

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.

sys_info fails to find mne-qt-browser

4 participants