Skip to content
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

New release 1.1.0 #362

Merged
merged 493 commits into from
Sep 19, 2023
Merged

New release 1.1.0 #362

merged 493 commits into from
Sep 19, 2023

Conversation

LilyAnderssonLee
Copy link
Contributor

PR checklist

  • This comment contains a description of changes (with reason).
  • If you've fixed a bug or added code that should be tested, add tests!
  • If you've added a new tool - have you followed the pipeline conventions in the contribution docs- [ ] If necessary, also make a PR on the nf-core/taxprofiler branch on the nf-core/test-datasets repository.
  • Make sure your code lints (nf-core lint).
  • Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
  • Usage Documentation in docs/usage.md is updated.
  • Output Documentation in docs/output.md is updated.
  • CHANGELOG.md is updated.
  • README.md is updated (including new tool citations and authors/contributors).

LilyAnderssonLee and others added 30 commits July 21, 2023 13:42
Change falco process into medium, solve memory issue
Remove meta clone everywhere for improved resume stability
Fix CENTRIFUGE header modification when --out-fmt turned on
The placement of the function is debatable, but to have its definition evaluated by nextflow, rather than pure Groovy makes it work.
Updated Bracken and Kraken2 database description to reflect the current contents on @BenLangmead database indexes. Also, required *.kraken from the expected files list for Bracken since they are no longer needed.
Updated Change Log to mention the usage update in PR #342
Copy link
Member

@jfy133 jfy133 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some responses to remaining commnetS!

- `<sample_id>.centrifuge.report.txt`: A classification report that summarises the taxonomic ID, the taxonomic rank, length of genome sequence, number of classified and uniquely classified reads
- `<sample_id>.centrifuge.results.txt`: A file that summarises the classification assignment for a read, i.e read ID, sequence ID, score for the classification, score for the next best classification, number of classifications for this read
- `<sample_id>.centrifuge.txt`: A Kraken2-style report that summarises the fraction abundance, taxonomic ID, number of k-mers, taxonomic path of all the hits in the centrifuge run for a given sample
- `<sample_id>.centrifuge.unmapped.fastq.gz`: FASTQ file containing all unmapped reads

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes,!

modules/local/kraken2_standard_report.nf Outdated Show resolved Hide resolved
subworkflows/local/shortread_fastp.nf Outdated Show resolved Hide resolved
subworkflows/local/standardisation_profiles.nf Outdated Show resolved Hide resolved
tower.yml Show resolved Hide resolved
@jfy133 jfy133 requested review from maxulysse and sofstam September 15, 2023 11:04
@LilyAnderssonLee
Copy link
Contributor Author

@maxulysse Could you approve the merge if you don't have any comments or reviews?

pattern: '*.fastq.gz',
enabled: params.save_analysis_ready_fastqs,
// Don't know why `!` doesn't work here, but `== false` makes it work...
saveAs: { ( params.perform_runmerging == false || ( params.perform_runmerging && !meta.is_multirun ) ) && !params.perform_shortread_hostremoval && !params.perform_shortread_complexityfilter && params.perform_shortread_qc && params.save_analysis_ready_fastqs ? it : null }
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😱

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know I know... I'll refactor in the future™️

withName: MALT_RUN {
tag = {"${meta.db_name}|${meta.id}"}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah cool, TIL :)

Comment on lines +130 to +132
// TODO: OPTIONAL, you can use nf-validation plugin to create an input channel from the samplesheet with Channel.fromSamplesheet("input")
// See the documentation https://nextflow-io.github.io/nf-validation/samplesheets/fromSamplesheet/
// ! There is currently no tooling to help you write a sample sheet schema
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems like template leftovers, but no big deal

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, leaving that as a reminder for the next release

@LilyAnderssonLee LilyAnderssonLee merged commit d1f82f3 into master Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

8 participants