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

Support Snakemake version 8 across core pathogen workflows #4

Open
1 of 5 tasks
victorlin opened this issue Jul 12, 2024 · 6 comments
Open
1 of 5 tasks

Support Snakemake version 8 across core pathogen workflows #4

victorlin opened this issue Jul 12, 2024 · 6 comments

Comments

@victorlin
Copy link
Member

victorlin commented Jul 12, 2024

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

@corneliusroemer
Copy link
Member

corneliusroemer commented Aug 22, 2024

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

@victorlin
Copy link
Member Author

Sure! That seems pretty low effort. I'll draft a docker-base PR now.

@victorlin
Copy link
Member Author

Test image is available as nextstrain/base:branch-victorlin-install-snakemake-8. It's not possible to use the proposed simple name nextstrain/base:snakemake8 without manually pushing another tag which could get out of sync with PR. The next simplest would be nextstrain/base:branch-snakemake8. Mine is lengthier due to branch naming habits, but copy/pasting this value is probably a good idea anyways.

@victorlin
Copy link
Member Author

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.

@genehack
Copy link
Contributor

genehack commented Aug 23, 2024

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)?

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.)

Currently all our pathogen repos pull the latest Docker image, so this would require either pinning specific runtime versions or a well-coordinated update.

This feels like a small piece of a larger conversation about long-term pathogen repo maintenance practices.

@tsibley
Copy link
Member

tsibley commented Aug 27, 2024

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.

@victorlin victorlin transferred this issue from another repository Nov 7, 2024
@victorlin victorlin transferred this issue from another repository Nov 7, 2024
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

No branches or pull requests

4 participants