Skip to content

Manuscript comments Tina - #550

Merged
jasmezz merged 3 commits into
paperfrom
paper-comments-tina
Sep 14, 2026
Merged

jasmezz merged 3 commits into
paperfrom
paper-comments-tina

Conversation

@jasmezz

@jasmezz jasmezz commented Sep 14, 2026

Copy link
Copy Markdown
Collaborator

This PR contains @warinner's manuscript improvements.

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/funcscan branch on the nf-core/test-datasets repository.
  • Make sure your code lints (nf-core pipelines lint).
  • Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
  • Check for unexpected warnings in debug mode (nextflow run . -profile debug,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).

warinner and others added 3 commits September 12, 2026 19:20
Editing of manuscript to reduce length, reduce repetition, and clarify arguments. Also, I made the order of the genes/functions consistent throughout the manuscript to match the title. They now always appear BGC first, then AMPs, then ARGs, then CAZymes/CGCs.
Co-authored-by: Jasmin Frangenberg <73216762+jasmezz@users.noreply.github.com>
@jasmezz
jasmezz merged commit a745dea into paper Sep 14, 2026
2 of 4 checks passed
@jasmezz
jasmezz deleted the paper-comments-tina branch September 14, 2026 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants