-
Notifications
You must be signed in to change notification settings - Fork 18
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
Breaking changes in Snakemake 8 #21
Comments
IntroductionI would like to start this process (initial work here), but I have a question about how to implement the changes required. I'm going to be using The current header should be changed from Options for UpdatesAppend documentation for snakemake-specific versionsIt would be relatively easy to copy-paste documentation, changing flags where required. However, this would result in double the amount of documentation that is required to keep up-to-date in the future # Snakemake <8.0
## Cancel running jobs with `--cluster-cancel`
(<8 docs here)
# Snakemake >8.0
## Cancel running jobs with `--cluster-generic-submit-cmd`
(>8 docs here) Place
|
Hi @jdblischak, I was using your
Everything did run smoothly under Cheers Bastian |
@bheimbu Is there any specific reason you need to use snakemake >8? If not, you should downgrade to the latest version of snakemake 7, eg |
@JoshLoecker thanks so much for volunteering to help with this! It is super appreciated, and I apologize for my delayed response. Ultimately I give you a lot of decision-making power since you are the one willing to work on it. I'm fine with your proposal of putting the new flag in parentheses and providing two config.yaml files. Though I will propose the alternative of having multiple branches. I can create the branches |
@JoshLoecker updates from my end: I created a branch v7 to preserve the snakemake <8 version of the documentation, pushed d6cbdbe to warn users in the README about the transition, and tagged v1.4.0 as the final release prior to migrating to snakemake 8. |
That sounds great! I will update my fork to reflect this |
Big thanks to @JoshLoecker for migrating the profile and docs to support Snakemake 8+! I just released v2.0.0 https://github.com/jdblischak/smk-simple-slurm/releases/tag/v2.0.0 |
Snakemake 8.0.0 introduced some breaking changes. The most obvious that would need to be updated are:
use-conda: True
would need to be updated tosoftware-deployment-method: conda
use-conda: True
+use-singularity: True
would need to be updated tosoftware-deployment-method: conda apptainer
But there are probably others. I don't have the time to dedicate to this at the moment, but I'm happy to review PRs from volunteers
The text was updated successfully, but these errors were encountered: