Thomas S. Binns

Results 108 comments of Thomas S. Binns

Hmm, seems like calling `pip install git+https://github.com/tsbinns/mne-python@complex_spectrum` ends up installing a dev branch of MNE 1.4

That looks better: ![image](https://github.com/user-attachments/assets/db11dd77-12af-4564-98bd-57527da366aa) Never would've figure that out. Thanks!

I fixed something for the Welch/Fourier mode I overlooked in my previous implementation when I was refactoring the code. Turns out that changed some of the connectivity values so the...

Will update spectrum tests next week with multivar. methods to make sure this gets coverage

Have expanded test coverage, should make things more robust.

Switching CI back to upstream since https://github.com/mne-tools/mne-python/pull/12747 was merged.

Makes sense, then the change is also super simple: https://github.com/sphinx-gallery/sphinx-gallery/blob/f066eb56a7c7c84f9b1835ff81dcabab8a80f798/sphinx_gallery/gen_rst.py#L394-L397 And in the example: https://github.com/sphinx-gallery/sphinx-gallery/blob/f066eb56a7c7c84f9b1835ff81dcabab8a80f798/examples/no_output/plot_raise_thumbnail.py#L1-L19 If people are happy with this, should the new variable be listed in the "Configurations...

> Yeah I think that makes sense! Sorry, just to clarify, is this a thumbs up just for the new solution or also for adding to the user guide?

Updated the name of the new example to better reflect what it now demonstrates. Also added an entry into the user guide explaining the new option.