-
Notifications
You must be signed in to change notification settings - Fork 828
Add sentieon star #1590
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
Add sentieon star #1590
Conversation
|
Warning Newer version of the nf-core template is available. Your pipeline is using an old version of the nf-core template: 3.3.1. For more documentation on how to update your pipeline, please see the nf-core documentation and Synchronisation documentation. |
|
maxulysse
left a comment
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.
Llooking good, @pinin4fjords covered most of the questions already.
There are some minor typos left
Co-authored-by: Jonathan Manning <[email protected]> Co-authored-by: Maxime U Garcia <[email protected]>
edmundmiller
left a comment
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.
Looks great to me!
Might have to revisit how we're testing Sentieon vs what we're supporting.
We could probably have a util for it in nf-core-utils 🤔
pinin4fjords
left a comment
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.
Nice clean solution, feel free to merge when you've sorted the tests, and sorry for my diversion ;-)
This PR add Sentieon support for STAR. The output of sentieon is equivalent to STAR. For this three modules where added:
Since Sentieon needs a number of things to be able to ping the license server, the respective scripts, and environment variables were added to :
bin/.githubAdditional tests to test these changes were added for the individual subworkflows
alignandrsem, as well as 2 new pipeline level tests, one expanding the default one (star & salmon) with sentieon enabled, one that is testing star_rsem directly.The usage docs on how to enable sentieon were added.
PR checklist
nf-core pipelines lint).nextflow run . -profile test,docker --outdir <OUTDIR>).nextflow run . -profile debug,test,docker --outdir <OUTDIR>).docs/usage.mdis updated.docs/output.mdis updated.CHANGELOG.mdis updated.README.mdis updated (including new tool citations and authors/contributors).