Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Don't warn when using decim with ICA.fit() on epochs - #10065

Merged
hoechenberger merged 2 commits into
mne-tools:mainfrom
hoechenberger:followup-of-10041
Dec 1, 2021
Merged

Don't warn when using decim with ICA.fit() on epochs#10065
hoechenberger merged 2 commits into
mne-tools:mainfrom
hoechenberger:followup-of-10041

Conversation

@hoechenberger

Copy link
Copy Markdown
Member

Follow-up to #10041

cc @agramfort

@hoechenberger hoechenberger added the backport-candidate on-merge: backport to maint/1.13 label Dec 1, 2021
@hoechenberger
hoechenberger marked this pull request as ready for review December 1, 2021 09:39
@agramfort

Copy link
Copy Markdown
Member

@hoechenberger please merge and backport

@hoechenberger
hoechenberger merged commit 113dafb into mne-tools:main Dec 1, 2021
@hoechenberger
hoechenberger deleted the followup-of-10041 branch December 1, 2021 11:30
hoechenberger added a commit to hoechenberger/mne-python that referenced this pull request Dec 1, 2021
* Don't warn when using decim with ICA.fit() on epochs

Follow-up to mne-tools#10041

* Fix tests
larsoner pushed a commit that referenced this pull request Dec 1, 2021
* decim is actually used for epochs (#10041)

* Don't warn when using decim with ICA.fit() on epochs (#10065)

* Don't warn when using decim with ICA.fit() on epochs

Follow-up to #10041

* Fix tests

Co-authored-by: Alexandre Gramfort <[email protected]>
@hoechenberger hoechenberger added backported and removed backport-candidate on-merge: backport to maint/1.13 labels Dec 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants