Daniel McCloy

Results 826 comments of Daniel McCloy

> @mscheltienne how shall we handle different datasets for testing? If I use `pytest.mark.parametrize("dataset", ["ca_208", "andy_101"])` and never mention `andy_101` explicitly, I see the following vulture error: `mne/io/ant/tests/test_ant.py:74: unused function...

This is actually quite a hard problem. Our website theme has good support for tables (and dataframes) already, but we're using a bunch of custom CSS instead so that the...

one idea that might work is to craft the CSS rules so that there are two co-extensive sets of rules: one set in `repr.css` and one in the main website...

here's what the repr HTML tables look like on built docs: https://output.circle-artifacts.com/output/job/df0a833c-b387-4034-a9e4-f3d59d80ef3f/artifacts/0/html/auto_tutorials/raw/20_event_arrays.html#sphx-glr-auto-tutorials-raw-20-event-arrays-py The buttons for showing bad channels are a bit wonky; I'll have to work on that. Pushing commits...

> @drammock I marked for 1.8, this is probably a blocker right? could also just backport it afterwards I guess? I will try to finish today but can't promise it.

OK here are some screenshots (hover effect does work in Jupyter light mode, I must have just wiggled the mouse when screenshotting): ### Sphinx ![built-docs-dark](https://github.com/user-attachments/assets/d870afc1-734d-49f0-b8fb-8e614dd5dd4a) ![built-docs](https://github.com/user-attachments/assets/d0926384-a684-46b8-80ac-1c23670cfb44) ### Jupyter ![juypter-dark](https://github.com/user-attachments/assets/758d1dc2-5b0f-4b36-9603-aef0b8f96074) ![jupyter-light](https://github.com/user-attachments/assets/785a0738-befe-4a9a-9dc9-fce183694000)...

> Aren't the carets in Jupyter and VS Code swapped between "opened" and "collapsed" state? oops yes, good catch.

@hoechenberger @larsoner this one is ready!

from past discussions: - [ ] PR reviews - [ ] obtaining funding - [ ] teaching MNE workshops

thank you for opening this. I think the docstring is confusing but technically not wrong: > None (default) will pick all channels does imply that bad channels will be included....