-
Notifications
You must be signed in to change notification settings - Fork 0
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
Support Snakemake version 8 across core pathogen workflows #4
Comments
Thanks @victorlin! How about we create a snakemake 8 image and do test runs with it in repos? Then we'll know what needs to change. I've added it as a task |
Sure! That seems pretty low effort. I'll draft a docker-base PR now. |
Test image is available as |
I'm wondering how we would follow through with a big change like this. Is it possible to update to be Snakemake 8 compatible and still work on Snakemake 7? Or are the two mutually exclusive (this is my impression)? Currently all our pathogen repos pull the latest Docker image, so this would require either pinning specific runtime versions or a well-coordinated update. |
Seems like confirming whether or not 8 is backcompat with 7 would be a first step? Has anybody built a list of changes? (That would also help with the inevitable "why are we doing this" question I expect.)
This feels like a small piece of a larger conversation about long-term pathogen repo maintenance practices. |
This, for sure. While the current practices worked well for a smaller number of production pathogens, they won't necessarily work as well for the larger number we're growing to. |
Currently, our managed runtimes come with Snakemake version 7 so our core pathogen workflows have been written to work with that.
Snakemake version 8 was released in December 2023.
It's known to not work in ncov, reported by an external user. Not sure about other repos as we haven't tried yet.
Tasks
docker.io/nextstrain/base:branch-victorlin-install-snakemake-8
--stats
when we upgrade to Snakemake v8 ncov-ingest#452The text was updated successfully, but these errors were encountered: