-
Notifications
You must be signed in to change notification settings - Fork 115
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
nf-core 3.3.0 release #734
Conversation
…subworkflow for alternative use of chopper for lambda-removal instead of nanolyse, and nanoq for longread filtering instead of filtlong
- Update modules - Update integration in mag and with other tools (bin_summary, gtdb-tk) - Update test - Update schema
Also, simplify bin_summary regarding bin qc
Co-authored-by: James A. Fellows Yates <[email protected]>
Post patch version bump
Add parameter to exclude unbinned data from post-binning steps
Bin QC Improvements
Fix input to single-end MEGAHIT runs
Add support for Prokka compliance mode
Important! Template update for nf-core/tools v3.1.0
Bump version for 3.3.0 release
|
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.
minor suggestions.
Looks good overall. I would aim to have to local modules & sub-wfs nf-tests by next release.
Co-authored-by: Sateesh_Peri <[email protected]>
Thank you! Slowly slowly getting there 😅 most of the local modules are being ripped and replaced, and I need pipeline tests done first becuase we are getting so many new contributors/ions, but I need to rejig the tests first , but that keeps getting pushed because we then get more bugs/contributors 😅 (I swear this pipeline is a full time job...) |
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 looked at all the changes except for the nf-core modules and subworkflows, which I expect to be exact copies from nf-core/modules.
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.
Thanks for the changes and explanations.
PR checklist
nf-core pipelines lint
).nextflow run . -profile test,docker --outdir <OUTDIR>
).nextflow run . -profile debug,test,docker --outdir <OUTDIR>
).docs/usage.md
is updated.docs/output.md
is updated.CHANGELOG.md
is updated.README.md
is updated (including new tool citations and authors/contributors).