Skip to content

Add MGnify amplicon tutorial #5292

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

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

RZ9082
Copy link
Contributor

@RZ9082 RZ9082 commented Sep 9, 2024

Adding a tutorial for MGnify amplicon v5.0 pipeline.

@paulzierep I look forward to your feedback :)

Copy link
Collaborator

@paulzierep paulzierep left a comment

Choose a reason for hiding this comment

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

This is great, and we should try to get it ready by GTA, I have only minor comments:

  • Can you explain more, for the individual steps, that you only need to run workflows if you have this kind of data, e.g. if you know your amplicons are 16S you do not need ITS workflow
  • Can you at the end propose how to adjust the workflow to users own data or update, like adjust fastp QC if needed with one or two examples, and also choose the newer DBs from v6 - those are already available right ?

@paulzierep
Copy link
Collaborator

@shiltemann we do not really understand the linting error, any idea, if possible we would like to add it for GTA if we can fix it by Sunday :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants