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

Update SPAdes 3.12.0 #38819

Merged
merged 2 commits into from
Jan 13, 2023
Merged

Update SPAdes 3.12.0 #38819

merged 2 commits into from
Jan 13, 2023

Conversation

kciy
Copy link
Contributor

@kciy kciy commented Jan 13, 2023

Using SPAdes 3.12.0 with a Python version >3.9 leads to the error described in this issue. Python versions 3.10 and above don't have the alias to the old collections class anymore (see this issue).

Setting the python version to below 3.10 fixes the problem for SPAdes 3.12.0 (it is anyways fixed in SPAdes >=3.15.4).

Copy link
Contributor

@wm75 wm75 left a comment

Choose a reason for hiding this comment

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

Careful work there @kciy. Thanks for the good explanation and helpful links!

@bgruening
Copy link
Member

@bioconda-bot please merge

@BiocondaBot
Copy link
Collaborator

I will attempt to upload artifacts and merge this PR. This may take some time, please have patience.

@BiocondaBot BiocondaBot merged commit 040a0db into bioconda:master Jan 13, 2023
cokelaer pushed a commit to cokelaer/bioconda-recipes that referenced this pull request Apr 28, 2023
Merge PR bioconda#38819, commits were: 
 * Merge branch 'master' into update-spades-3.12.0
 * pin python version below 3.10
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

Successfully merging this pull request may close these issues.

None yet

4 participants