-
Notifications
You must be signed in to change notification settings - Fork 7
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
Reconfigure how containers are used in nextflow pipelines #442
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quite a neat update, lots to learn 🤓
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was also wondering if we should log the usage of -profile
somewhere, maybe we can add it in nf-schema file so it shows as an optional parameter while running the pipeline?
Expand versioning regex for 10+ versions Co-authored-by: J. Alvarez-Jarreta <[email protected]>
Co-authored-by: J. Alvarez-Jarreta <[email protected]>
Co-authored-by: J. Alvarez-Jarreta <[email protected]>
Co-authored-by: J. Alvarez-Jarreta <[email protected]>
Co-authored-by: Disha Lodha <[email protected]>
Co-authored-by: Disha Lodha <[email protected]>
Nextflow has a command to show the config process directives and profiles etc. In the pipeline workflow folder of each pipeline, if you do
|
Co-authored-by: Disha Lodha <[email protected]>
Co-authored-by: J. Alvarez-Jarreta <[email protected]>
Co-authored-by: J. Alvarez-Jarreta <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One tiny bug to be addressed. Since there is no Python code involvement, I'm happy to merge with pylint
failing (I believe this has been fixed in main
already). However, the new sub-pipeline is also failing, and that may need to be fixed.
Co-authored-by: J. Alvarez-Jarreta <[email protected]>
bd703e2
to
47fd389
Compare
The basis of this PR is to integrate the use of the newly developed genomio-container (Docker) to be used in:
List of changes:
Addition: