-
Notifications
You must be signed in to change notification settings - Fork 805
Dev -> Master for 3.21.0 #1607
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
Dev -> Master for 3.21.0 #1607
Conversation
Bump version back to dev after release 3.20.0
Enable BAM input for RSEM
Fix default for umi_discard_read to prevent validation errors in Platform
Bump version to 3.21.0 ahead of release
|
| star_rsem/*.{genes,isoforms}.results | ||
| star_rsem/log/*.log | ||
| star_salmon/log/*.Log.{final.out,out,progress.out} | ||
| star_{salmon,rsem}/log/*.Log.{final.out,out,progress.out} |
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.
| star_{salmon,rsem}/log/*.Log.{final.out,out,progress.out} | |
| star_{rsem,salmon}/log/*.Log.{final.out,out,progress.out} |
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.
obvious nitpick
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'll see if there's other feedback before I trigger the ci just for that ;-)
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 had to wrap my head around the changes from RSEM since I didn't follow the PR that was changing that, but all good to me
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.
Just a small suggestion regarding the changelog, otherwise LGTM 🚀
This release is dedicated almost entirely to two recent PRs which add BAM input functionality to RNAseq, and move the STAR alignment out of the RSEM process so that it's more visible and controllable. Everything else is just fallout from that- e.g. we have to configure STAR differently if its results are going to Salmon or STAR.
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).