search

mne-tools / mne-bids-pipeline

Automatically process entire electrophysiological datasets using MNE-Python.
https://mne.tools/mne-bids-pipeline/
BSD 3-Clause "New" or "Revised" License
134 stars 65 forks source link

DOC: Mark steps as optional in flowchart #861

Closed larsoner closed 5 months ago

larsoner commented 5 months ago

@larsoner is the 'temporal regression for artifact removal' a default step? I am not aware that I am using it.

If not, I think it should be marked as optional.

Originally posted by @SophieHerbst in https://github.com/mne-tools/mne-bids-pipeline/issues/860#issuecomment-1963769622

larsoner commented 5 months ago

In some sense almost all steps are optional. Like if you set l_freq = None (default) and h_freq = None (not default!) there is no frequency filtering, spatial_filter = None (default) skips SSP and ICA, etc. I'll add a note to #865 along these lines. After that I think that documenting better which config options affect which steps is probably the best we can do (?), in which case this would be a dup of #862