-
Notifications
You must be signed in to change notification settings - Fork 68
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
BUG: Fix several bugs #839
Conversation
Thanks @larsoner! I think the value of the "reject" parameter should be included via upstream changes to mne.Report.add_epochs(). But we can merge here and move the feature to MNE-Python at a later time. Would be good to create an issue as a reminder |
Found another bug! |
Before merging …
docs/source/changes.md
)I went to work on #831 but wanted to sort out a few little other bugs (and one minor enhancement) first:
Fix bug where using
mf_reference_run = None
could change the empty room file matched from run-to-run ofmne_bids_pipeline
due to use ofset
, whose order is not guaranteed. @hoechenberger I suspect this could be the actual root cause of Caching seems not to be working on macOS #814Output the
reject
params in the reject step -- though this can be set in the config, it's sometimes calculated on-the-fly byautoreject
and not saved anywhere. Even in the manualdict
case it's nice to have it near the drop logs in the report in case you want to think about adjusting your thresholds:Fix bug where
reject
was getting restricted incorrectly for M/EEG channelsPrefer
stat
tolstat
because I'm pretty sure we don't want to use the symlink's ownmtime
but rather themtime
of whatever it points to. (This is apparently the only difference betweenlstat
andstat
.) Doubt anyone has actually hit this so no changelog needed, but better to have it in there I think.Fix bug with
-regress
saving where instead of writing one for each run, a single filename was written to by every run (within-release change so no changelog needed).Slightly better report tags for the cleaned epochs to make it clear that what's happened is the
reject
step