-
Notifications
You must be signed in to change notification settings - Fork 20
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
Snakemake module #367
base: main
Are you sure you want to change the base?
Snakemake module #367
Conversation
@FabianHofmann thanks for setting up the PR! Do you need any support, or is any review required yet? If so, just let me know. |
Thank you @energyLS, I am still playing a little bit, but I am making good progress. A short question regarding the log. Is there any way to suppress the quite verbose two outputs:
Looked a little but so far I could not find which rule triggers these logs. |
@ekatef as you were having issues with the environment. In this PR, the CI for both Mac and Ubuntu is working. I think it is mainly because the CI uses |
Hey @FabianHofmann, thanks for notifying! I'm still puzzled about the environment troubles for mac... We had an impression that the troubles are caused by a transition from So, great news that the CI is working now which seems to be the case also with |
Yes, it can be a real pain sometimes... fingers crossed that this will resolve also in the long term :) |
This change proposes to replace snakemake the subworkflow approach with a submodule approach. As discussed this is sill in the testing phase (CI will likely fail in the first iterations).
Closes # (if applicable).
Changes proposed in this Pull Request
Checklist
envs/environment.yaml
andenvs/environment.docs.yaml
.config.default.yaml
,config.tutorial.yaml
, andtest/config.test1.yaml
.doc/configtables/*.csv
and line references are adjusted indoc/configuration.rst
anddoc/tutorial.rst
.doc/release_notes.rst
is amended in the format of previous release notes, including reference to the requested PR.