-
Notifications
You must be signed in to change notification settings - Fork 33
Add InterProScan to Pipeline and integrate in AMPcombi #428
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
Changes from all commits
86592d9
491f25d
bbd456e
5b5cb3e
d8c5bf2
fd1ef46
b54f1ea
fee3adb
8b44ed5
ed81b0b
0340ba9
7e1f164
5be17ef
b782b54
e58b322
310a0be
8c8aeaf
804da18
186d351
f88fe9c
5d1f4b7
a7e681d
f14990f
9ecac2e
bdf0a98
b10dd7d
a54d7aa
5dd1a8f
d4c6ce6
f68d316
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -91,7 +91,7 @@ process { | |
| ].join(' ').trim() | ||
| } | ||
|
|
||
| withName: SEQKIT_SEQ { | ||
| withName: SEQKIT_SEQ_LENGTH { | ||
| ext.prefix = { "${meta.id}_long" } | ||
| publishDir = [ | ||
| path: { "${params.outdir}/bgc/seqkit/" }, | ||
|
|
@@ -104,6 +104,45 @@ process { | |
| ].join(' ').trim() | ||
| } | ||
|
|
||
| withName: SEQKIT_SEQ_FILTER { | ||
| ext.prefix = { "${meta.id}_cleaned.faa" } | ||
| publishDir = [ | ||
| path: { "${params.outdir}/protein_annotation/interproscan/" }, | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Are we sure we want the output in If not, what do you think of renaming the
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The other annotation tools in the annotation workflow are annotating CDS which also can be proteins. I would leave the annotation workflow as is because thats the baseline annotation step of the pipeline. This annotation step is more of an accessory annotation to the pipeline if the user wants more information from diff DB and (1) its technically not correct protein_annotation because those are nnot necessary proteins and (2) the plan is that we add more to this workflow (e.g. the functional annotation of those CDS) so protein annotation will no longer be valid.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The ouput im not sure if its a good idea to add it in the same folder because those are two diff workflows
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Okay. No strong opinion from my side. I still don't find the naming super intuitive, but it can be. Or why not |
||
| mode: params.publish_dir_mode, | ||
| enabled: { params.run_protein_annotation }, | ||
| saveAs: { filename -> filename.equals('versions.yml') ? null : filename } | ||
| ] | ||
| ext.args = [ | ||
| "--gap-letters '* \t.' --remove-gaps" | ||
| ].join(' ').trim() | ||
| } | ||
|
|
||
| withName: INTERPROSCAN_DATABASE { | ||
| publishDir = [ | ||
| path: { "${params.outdir}/databases/interproscan/" }, | ||
| mode: params.publish_dir_mode, | ||
| enabled: params.save_db, | ||
| saveAs: { filename -> filename.equals('versions.yml') ? null : filename } | ||
| ] | ||
| } | ||
|
|
||
| withName: INTERPROSCAN { | ||
| ext.prefix = { "${meta.id}_interproscan.faa" } | ||
Darcy220606 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| publishDir = [ | ||
| path: { "${params.outdir}/protein_annotation/interproscan/" }, | ||
| mode: params.publish_dir_mode, | ||
| enabled: params.run_protein_annotation, | ||
| saveAs: { filename -> filename.equals('versions.yml') ? null : filename } | ||
| ] | ||
| ext.args = [ | ||
| "--applications ${params.protein_annotation_interproscan_applications}", | ||
| params.protein_annotation_interproscan_enableprecalc ? '' : '--disable-precalc', | ||
| '--disable-residue-annot', | ||
| '--enable-tsv-residue-annot', | ||
| "--formats tsv" | ||
| ].join(' ').trim() // Warning: Do not disable the flags "--enable-tsv-residue-annot" and "--formats tsv"! This would cause a run failure because the format of the resulting files would no longer be adequate for parsing by AMPcombi2. | ||
| } | ||
|
|
||
| withName: PROKKA { | ||
| ext.prefix = { "${meta.id}_prokka" } | ||
| publishDir = [ | ||
|
|
@@ -687,7 +726,7 @@ process { | |
|
|
||
| withName: AMP_DATABASE_DOWNLOAD { | ||
| publishDir = [ | ||
| path: { "${params.outdir}/databases/${params.amp_ampcombi_db}" }, | ||
| path: { "${params.outdir}/databases/ampcombi/" }, | ||
| mode: params.publish_dir_mode, | ||
| enabled: params.save_db, | ||
| saveAs: { filename -> filename.equals('versions.yml') ? null : filename }, | ||
|
|
||
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.