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

Use pick method instead of pick_channels as it is now an MNE legacy function #118

Open
alexenge opened this issue Jun 30, 2023 · 0 comments
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@alexenge
Copy link
Owner

Running the pipeline with a recent version of MNE (e.g., 1.4) prints the following warning a couple of times:

NOTE: pick_channels() is a legacy function. New code should use inst.pick(...).

To resolve this, it should be enough to replace all something.pick_channels(...) calls with something.pick(...). Apparently, the standalone function (not the method) mne.pick_channels remains okay and does not need to be changed.

The new pick method is available since MNE 0.24, so it should be fairly safe to use since probably nobody using the pipeline will be on an older MNE version.

@alexenge alexenge added bug Something isn't working good first issue Good for newcomers labels Jun 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant