Eric Larson

Results 147 issues of Eric Larson

### Checklist - [X] I added a descriptive title - [X] I searched open reports and couldn't find a duplicate ### What happened? - When doing a system-wide install on...

type::bug

> So would would have been a viable solution for @SophieHerbst now? Simply re-run the entire `preprocessing` pipeline and rely on caching to skip e.g. filtering etc? We don't have...

### Before merging … - [x] Changelog has been updated (`docs/source/changes.md`) 1. Moved all ICA reports to the main report: ![image](https://github.com/mne-tools/mne-bids-pipeline/assets/2365790/ab1ab2cb-89ee-41f1-b759-d6585a34e448) 2. Added docs suggesting manual intervention is required (maybe...

When looking at https://github.com/mne-tools/mne-bids-pipeline/pull/914 it occurred to me that command-line options like `--no-cache` are not documented anywhere as far as I can tell. This could perhaps be added as another...

In https://mne.tools/mne-bids-pipeline/stable/settings/preprocessing/autobads.html it says: > Warning > > This functionality will soon be removed from the pipeline, and will be integrated into MNE-BIDS. This is from 3 years ago --...

Currently we parallelize over subjects and runs. This leads to stuff where in parallel mode run 4 can be processed faster than run 1, and thus appears in report first....

> this is the third time I've seen the same inline "filt-or-regress" if-else. Makes me worry about forgetting to copy it where it's needed, when adding new code. Not sure...

One solution that should work for this would be something like: ``` mf_bads: Literal["run", "union"] = "run" ``` where changing to `"union"` it will use the union of all bads...

EOSS4

I have a dataset that I'm BIDSifying. For each subject we have the following based on how I'm BIDSifying the data: 1. Resting state data (`task="rest"`) 2. One run of...

We might consider doing a new publication (JOSS?) to document the 1.0 release. The pipeline has come a long way since MNE-Study-template, and some contributors (e.g., @hoechenberger) are not on...