Skip to content

[MRG] fix: Cap pooch dependency version#1241

Merged
ntolley merged 1 commit intojonescompneurolab:masterfrom
asoplata:fix/cap-pooch-version
Feb 11, 2026
Merged

[MRG] fix: Cap pooch dependency version#1241
ntolley merged 1 commit intojonescompneurolab:masterfrom
asoplata:fix/cap-pooch-version

Conversation

@asoplata
Copy link
Collaborator

Currently, both the script and notebook versions of our example file examples/workflows/plot_simulate_somato.py fail because of an upstream breakage in how the MNE sample data is downloaded and unpacked (see here fatiando/pooch#518
This change fixes the issue by "capping" the Pooch version to a prior version which does not contain the bug.

Currently, both the script and notebook versions of our example file
`examples/workflows/plot_simulate_somato.py` fail because of an upstream
breakage in how the MNE sample data is downloaded and unpacked (see here
fatiando/pooch#518
This change fixes the issue by "capping" the Pooch version to a prior
version which does not contain the bug.
Copy link
Collaborator

@ntolley ntolley left a comment

Choose a reason for hiding this comment

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

Shall we look into undoing this when the bug is patched on their end?

I see that you might have found the solution for pooch so I think you'll be in the know when it's ready to retry

In any case I've verified on the docs that the somato example is functioning again so good to merge!

@ntolley ntolley merged commit 6a04c4c into jonescompneurolab:master Feb 11, 2026
11 checks passed
@asoplata
Copy link
Collaborator Author

Forgot to mention: This was actually the original cause of the problem in #1237, not the Python version itself.

@ntolley Yes absolutely. I submitted an upstream patch to pooch ( here fatiando/pooch#518 ). Additionally, it turns out that MNE had already solved this problem on their development branch, but not yet made a new release that includes it mne-tools/mne-python#13648 . We can revert this dependency-cap once either of those code changes are pushed publicly to a new version in either.

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

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

2 participants