Skip to content

Conversation

@jshanna100
Copy link

This adds PyPREP bad channel detection functionality to the first data quality step in the mne-bids-pipeline, just after MEG maxfilter based detection

@jshanna100 jshanna100 requested a review from behinger December 5, 2025 13:46
@jshanna100 jshanna100 self-assigned this Dec 5, 2025
# run the ransac algo
pyprep_by_ransac: bool = False
# params for ransac, defaults to empty dict, i.e. pyprep defaults
pyprep_by_ransac_params: dict = dict()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ultimately we should have a useful default configuration. maybe the one defined in the original github-issue on mne-bids-pipeline is reasonable?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

he seems to just use the defaults?
mne-tools#835

Copy link
Member

@behinger behinger Dec 16, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that's too bad then, if it fails misserably for us :(

We have to find our own defaults then.

@behinger behinger changed the title First commit PyPREP bad channel detectgion Dec 17, 2025
@behinger behinger changed the title PyPREP bad channel detectgion PyPREP bad channel detection Dec 17, 2025
@behinger
Copy link
Member

is there a report already on which function found which channel as bad?

@behinger
Copy link
Member

please note that we (finally) created a new fork, which we will try to maintain in parallel to the official mne-bids-pipeline

https://github.com/s-ccs/mne-bids-eyetracking-pipeline/

We can finish the PR here, and then merge it via https://github.com/s-ccs/mne-bids-eyetracking-pipeline/pull/2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants