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

Python 3.13 - ModuleNotFoundError: No module named 'pipes' #33

Open
rpetit3 opened this issue Nov 4, 2024 · 5 comments
Open

Python 3.13 - ModuleNotFoundError: No module named 'pipes' #33

rpetit3 opened this issue Nov 4, 2024 · 5 comments
Labels
bug Something isn't working

Comments

@rpetit3
Copy link
Owner

rpetit3 commented Nov 4, 2024

I will need to make some fixes on the bioconda recipe. But in case you get the following erro

ModuleNotFoundError: No module named 'pipes'

The solution is to downgrade your Python version to 3.12 or earlier.

@rpetit3 rpetit3 added the bug Something isn't working label Nov 4, 2024
@rpetit3 rpetit3 pinned this issue Nov 4, 2024
@mbhall88
Copy link
Contributor

mbhall88 commented Nov 4, 2024

Weird. We don't have a dependency on a module called pipes do we?

@rpetit3
Copy link
Owner Author

rpetit3 commented Nov 4, 2024

Yep we do (and its not in the pyproject .toml which is another issue itself haha) , https://github.com/rpetit3/fastq-dl/blob/master/fastq_dl/utils.py#L11

I've submitted a PR here conda-forge/executor-feedstock#8 , based on this PR xolox/python-executor#22

Executor is a package I've used for years for simplifying the usage of subprocess. We can reconsider its usage going forward, but if the PR fixes it (substitutes pipes for shlex) then I think we just leave it as is.

@mbhall88
Copy link
Contributor

mbhall88 commented Nov 5, 2024

yeah I was going to say subprocess, which is in the standard library is pretty easy to work with so I think we could very easily transition to that.

@rpetit3
Copy link
Owner Author

rpetit3 commented Dec 16, 2024

Heads up planning to work on dropping the wrapper and just going with subprocess directly

@apcamargo
Copy link

I've been using sh for a while now to simplify calling external programs. I guess it could replace Executor.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants