-
Notifications
You must be signed in to change notification settings - Fork 23
Add support for presupplied annotation files (FAA + GFF or FAA + GBK) #340
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
Closed
Closed
Changes from 45 commits
Commits
Show all changes
54 commits
Select commit
Hold shift + click to select a range
e280092
Start modifying samplesheet check (untested)
jfy133 9fbf52c
Made the samplesheet work if columns are existing (python error if th…
jfy133 cf2b8b7
Continue work
jfy133 80106d8
Apply suggestions from code review
jfy133 1b60b24
Get most of the log working, needs more testing (particularly non FAA…
jfy133 348b16c
Merge branch 'dev' into presupplied-orfs
jfy133 43ee615
Sync latest dev changes from annotation into workflow
jfy133 183b81a
Update all modules to get right container version and also pyRodigal …
jfy133 258a49a
Merge branch 'presupplied-orfs' of github.com:nf-core/funcscan into p…
jfy133 a7b03a7
Add test nothing config
jfy133 29670ab
Merge branch 'dev' into presupplied-orfs
jfy133 ac0a25d
Get back to previous starting point before bad merge removed old changes
jfy133 da993dd
Merge branch 'dev' into presupplied-orfs
jfy133 4370106
Refactor - have amp/arg working. Includes better fargene tagging
jfy133 7eb5bed
Have it working
jfy133 28a50ee
Add test profile and docs
jfy133 9caae3e
Include preanntotaed files in one of the CI runs
jfy133 23e3929
Fix prettier linting
jfy133 663a7e9
Fix ci command
jfy133 6177c90
Fix BAKTA to multiqc channel name
jfy133 bf0f572
Add a preannotated test to BGC workflows
jfy133 ea555ab
Make preannotated bgc config accesible
jfy133 80c5b0c
Install newer version of antismash to see if it'll work with the GFF …
jfy133 106b76e
Use correct dummy files
jfy133 c25cab1
Add warning about Prokka GBK/GFF
jfy133 89fdf9a
Merge branch 'dev' into presupplied-orfs
jasmezz f9f808d
Wrapping my head around it
jasmezz 9b483ac
Excluded GFF support, fixed multiqc report, update variables etc.
jasmezz 55224f1
Merge branch 'dev' into presupplied-orfs
jasmezz 43e77cf
Update usage docs and samplesheet
jasmezz 311f77c
Update modules.config, fix linting, variable typos
jasmezz 2ac179d
Fix variable typos, fix multiqc channel for bakta
jasmezz 212ce0c
Fix linting
jasmezz 15f9fbf
Prefer pyrodigal in tests, add warning when prodigal + antismash are …
jasmezz a8716f8
Apply suggestions from code review
jasmezz fd52fee
Apply suggestions from code review, fix linting
jasmezz 4e0a61f
Change feature to gbk, remove gff from docs
jasmezz de6bd40
Merge branch 'dev' into presupplied-orfs
jasmezz b5fc8f4
Fix "feature" renaming to "gbk"
jasmezz 95f8fb5
Fix linting
jasmezz de0a7bf
Merge branch 'dev' into presupplied-orfs
jasmezz bf43049
Fix variables
jasmezz 0d2ef7c
Fix channels, missing warnin/docs about no splitting for preanno
jfy133 7ed3594
Use correct GBK channel
jfy133 619479a
Add log warning when BGC and preannotated input
jfy133 85c4359
Start trying to fix taxonomy, not working yet as MMSEQS_TAXONOMYDB no…
jfy133 b11bed1
Add more GBK/GBFF updates
jfy133 800dff9
Remove dumps
jfy133 58787f6
Merge branch 'dev' into presupplied-orfs
jfy133 50aa076
Only do splitting when BGC workflow executed
jfy133 c419ce9
Fix taxonomy workflow from possibly getting async between two input c…
jfy133 d1d0177
Fix prokka annotation MQC collection
jfy133 8f1c7ba
Fix linting
jfy133 2d8b238
Make it so deepBGC actually produces otutput, and START send only lon…
jfy133 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,4 @@ | ||
sample,fasta | ||
sample_1,https://raw.githubusercontent.com/nf-core/test-datasets/funcscan/wastewater_metagenome_contigs_1.fasta.gz | ||
sample_2,https://raw.githubusercontent.com/nf-core/test-datasets/funcscan/wastewater_metagenome_contigs_2.fasta.gz | ||
sample,fasta,protein,gbk | ||
sample_1,https://raw.githubusercontent.com/nf-core/test-datasets/funcscan/wastewater_metagenome_contigs_1.fasta.gz,https://raw.githubusercontent.com/nf-core/test-datasets/funcscan/wastewater_metagenome_contigs_prokka_1.faa,https://raw.githubusercontent.com/nf-core/test-datasets/funcscan/wastewater_metagenome_contigs_prokka_1.gbk | ||
sample_2,https://raw.githubusercontent.com/nf-core/test-datasets/funcscan/wastewater_metagenome_contigs_2.fasta.gz,https://raw.githubusercontent.com/nf-core/test-datasets/funcscan/wastewater_metagenome_contigs_prokka_2.faa.gz,https://raw.githubusercontent.com/nf-core/test-datasets/funcscan/wastewater_metagenome_contigs_prokka_2.gbk.gz | ||
sample_3,https://raw.githubusercontent.com/nf-core/test-datasets/funcscan/wastewater_metagenome_contigs.fasta |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,32 @@ | ||
/* | ||
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | ||
Nextflow config file for running minimal tests | ||
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | ||
Defines input files and everything required to run a fast and simple pipeline test. | ||
|
||
Use as follows: | ||
nextflow run nf-core/funcscan -profile test,<docker/singularity> --outdir <OUTDIR> | ||
|
||
---------------------------------------------------------------------------------------- | ||
*/ | ||
|
||
params { | ||
config_profile_name = 'Test profile - preannotated input' | ||
config_profile_description = 'Minimal test dataset to check pipeline function' | ||
|
||
// Limit resources so that this can run on GitHub Actions | ||
max_cpus = 2 | ||
max_memory = '6.GB' | ||
max_time = '6.h' | ||
|
||
// Input data | ||
input = 'https://raw.githubusercontent.com/nf-core/test-datasets/funcscan/samplesheet_preannotated.csv' | ||
amp_hmmsearch_models = 'https://raw.githubusercontent.com/nf-core/test-datasets/funcscan/hmms/mybacteriocin.hmm' | ||
|
||
annotation_tool = 'pyrodigal' | ||
|
||
run_arg_screening = true | ||
arg_fargene_hmmmodel = 'class_a,class_b_1_2' | ||
|
||
run_amp_screening = true | ||
} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,31 @@ | ||
/* | ||
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | ||
Nextflow config file for running minimal tests | ||
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | ||
Defines input files and everything required to run a fast and simple pipeline test. | ||
|
||
Use as follows: | ||
nextflow run nf-core/funcscan -profile test_bgc,<docker/singularity> --outdir <OUTDIR> | ||
|
||
---------------------------------------------------------------------------------------- | ||
*/ | ||
|
||
params { | ||
config_profile_name = 'BGC test profile - preannotated input BGC' | ||
config_profile_description = 'Minimal test dataset to check BGC workflow function' | ||
|
||
// Limit resources so that this can run on GitHub Actions | ||
max_cpus = 2 | ||
max_memory = '6.GB' | ||
max_time = '6.h' | ||
|
||
// Input data | ||
input = 'https://raw.githubusercontent.com/nf-core/test-datasets/funcscan/samplesheet_preannotated.csv' | ||
bgc_hmmsearch_models = 'https://raw.githubusercontent.com/antismash/antismash/fd61de057e082fbf071732ac64b8b2e8883de32f/antismash/detection/hmm_detection/data/ToyB.hmm' | ||
|
||
annotation_tool = 'pyrodigal' | ||
|
||
run_arg_screening = false | ||
run_amp_screening = false | ||
run_bgc_screening = true | ||
} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Is that in yet?
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.
Yes, need to pull dev