Skip to content
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

mne.filter.notch_filter default notch_widths too low? #13103

Open
SophieHerbst opened this issue Feb 4, 2025 · 1 comment
Open

mne.filter.notch_filter default notch_widths too low? #13103

SophieHerbst opened this issue Feb 4, 2025 · 1 comment

Comments

@SophieHerbst
Copy link
Contributor

Hello,
when applying a notch filter to EEG data without setting the notch width (stop band width), the default is frequencies / 200. For typical 50 Hz line noise, this results in 0.25 Hz, which appears very narrow. The attached plot shows that the result is not satisfactory.
I think it would be better to set a fixed band width, e.g. 1 Hz for all frequencies.
What do you think?
Sophie

Image
@larsoner
Copy link
Member

larsoner commented Feb 6, 2025

Yeah that probably is too narrow. It would probably be an annoying deprecation but probably worth it. I think so far I always set it to something else because the default is not good enough

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

No branches or pull requests

2 participants