Eric Larson

Results 1455 comments of Eric Larson

See also https://github.com/mne-tools/mne-python/issues/12980. I think the class of possible things to simulate is quite large... maybe now is a good time to split off so things like this can be...

Hah yes this seems like an internal confusion with how `mask` is interpreted. We should do with the labels whatever we do with the sensors themselves (and hopefully this is...

Yeah it might be better to check if the bound is less than the current `info["highpass"]`. Safest would be to make it an error if it is. We could be...

@Cathaway can you share the problematic file? cc @scott-huberty

Personally I don't use the diagnostic plots and just inspect reports so I don't have a lot of first-hand insight here. But what you suggest adding seems to make sense,...

The way I typically try to silence these is by fixing the upstream package and hoping they release :)

I think currently the way to know what is done is to look at the code. And to be able to answer your question it's what I would have to...

... and it would be awesome if you are interested in working on this, happy to look at a PR or give more guidance/ideas!

We currently don't really have a public API for this sort of thing. Looking at what the command-line `mne_bids_pipeline` actually invokes: https://github.com/mne-tools/mne-bids-pipeline/blob/cce897d3ac3324a6ab5fbd5f7333e1df3c61f919/mne_bids_pipeline/_main.py#L17 The two parts I think you need are...

I would expect some potential deviation due to having to format the data properly, but these errors should be small. Can you use something like `np.testing.assert_allclose(edffile.get_data(), subchannels.get_data(), atol=1e-14, rtol=1e-6)` and...