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

Small updates to schema JSON to remove false and "None" defaults. #302

Merged
merged 2 commits into from
Aug 9, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- [#297](https://github.com/nf-core/funcscan/pull/297) Added check for empty annotation files prior going into screening. (❤️ to @alexhbnr for requesting, added by @jfy133)
- [#299](https://github.com/nf-core/funcscan/pull/299) Fixed pigz error with symlinks in Pyrodigal. (by @jasmezz)
- [#300](https://github.com/nf-core/funcscan/pull/300) Fixed wrong Pyrodigal channels being submitted to antiSMASH. (reported by Till Bayer, fix by @jasmezz)
- [#302](https://github.com/nf-core/funcscan/pull/302) Removed trouble-causing default parameters in json schema. (by @robsyme)

### `Dependencies`

Expand Down
23 changes: 3 additions & 20 deletions nextflow_schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,6 @@
"properties": {
"annotation_bakta_db_localpath": {
"type": "string",
"default": "None",
"fa_icon": "fas fa-database",
"description": "Specify a path to BAKTA database.",
"help_text": "Specify a path to a database that is prepared in a BAKTA format."
Expand Down Expand Up @@ -251,7 +250,7 @@
"default": "Bacteria",
"fa_icon": "fab fa-accusoft",
"description": "Specify the kingdom that the input represents.",
"help_text": "Specifies the kingdom that the input sample is derived from and/or you wish to screen for\n\n> \u26a0\ufe0f Prokka cannot annotate Eukaryotes.\n\nFor more information please check Prokka [documentation](https://github.com/tseemann/prokka).\n\n> Modifies tool parameter(s):\n> - Prokka: `--kingdom`",
"help_text": "Specifies the kingdom that the input sample is derived from and/or you wish to screen for\n\n> ⚠️ Prokka cannot annotate Eukaryotes.\n\nFor more information please check Prokka [documentation](https://github.com/tseemann/prokka).\n\n> Modifies tool parameter(s):\n> - Prokka: `--kingdom`",
"enum": ["Archaea", "Bacteria", "Mitochondria", "Viruses"]
},
"annotation_prokka_gcode": {
Expand All @@ -272,7 +271,7 @@
},
"annotation_prokka_evalue": {
"type": "number",
"default": 1e-6,
"default": 0.000001,
"description": "Minimum e-value cut-off.",
"help_text": "Specifiy the minimum e-value used for filtering the alignment hits.\n\nFor more information please check Prokka [documentation](https://github.com/tseemann/prokka).\n\n> Modifies tool parameter(s):\n> - Prokka: `--evalue`",
"fa_icon": "fas fa-sort-amount-down"
Expand Down Expand Up @@ -301,7 +300,6 @@
"annotation_prokka_centre": {
"type": "string",
"description": "Sequencing centre ID.",
"default": "None",
"fa_icon": "fas fa-map-marker-alt",
"help_text": "Add the sequencing center ID used in generating the raw sequences. This flag is typically requested in combination with the `--compliant` flag when contigs need to be renamed due to non-conforming contig headers. For more information please check Prokka [documentation](https://github.com/tseemann/prokka). \n\n> Modifies tool parameter(s):\n> - Prokka: `--centre`"
},
Expand Down Expand Up @@ -471,7 +469,6 @@
},
"amp_hmmsearch_models": {
"type": "string",
"default": "None",
"description": "Specify path to the AMP hmm model file(s) to search against. Must have quotes if wildcard used.",
"help_text": "HMMSearch performs biosequence analysis using profile hidden Markov Models.\nThe models are specified in`.hmm` files that are specified with this parameter\n\ne.g. \n\n```\n--amp_hmmsearch_models '/<path>/<to>/<models>/*.hmm'\n```\n\nYou must wrap the path in quotes if you use a wildcard, to ensure Nextflow expansion _not_ bash!\n\nFor more information check HMMER [documentation](http://hmmer.org/).",
"fa_icon": "fas fa-layer-group"
Expand Down Expand Up @@ -524,7 +521,6 @@
"type": "string",
"description": "Path to AMPcombi reference database directory (DRAMP).",
"help_text": "AMPcombi uses the 'general AMPs' dataset of the (DRAMP database)[http://dramp.cpu-bioinfor.org/downloads/] for taxonomic classification. If you have a local version of it, you can provide the path to the folder containing the reference database files:\n1. a fasta file with a `.fasta` file extension\n2. the corresponding table with with functional and taxonomic classifications in `.tsv` file extension.\n\nFor more information check AMPcombi [documentation](https://github.com/Darcy220606/AMPcombi).",
"default": "None",
"fa_icon": "fas fa-address-book"
},
"amp_ampcombi_cutoff": {
Expand Down Expand Up @@ -552,7 +548,6 @@
},
"arg_amrfinderplus_db": {
"type": "string",
"default": "None",
"fa_icon": "fas fa-layer-group",
"help_text": "Specify the path to a local version of the ARMFinderPlus database. If no input is given, the pipeline will download the database for you.\n\n See the nf-core/funcscan usage [documentation](https://nf-co.re/funcscan/usage) for more information.",
"description": "Specify the path to a local version of the ARMfinderPlus database."
Expand Down Expand Up @@ -609,7 +604,6 @@
},
"arg_deeparg_data": {
"type": "string",
"default": "None",
"fa_icon": "fab fa-deezer",
"description": "Specify the path to the DeepARG database.",
"help_text": "Specify the path to a local version of the DeepARG database (see the pipelines' usage [documentation](https://nf-co.re/funcscan/usage)). If no input is given, the module will download the database for you, however this is not recommended, as the database is large and this will take time."
Expand Down Expand Up @@ -855,22 +849,20 @@
"type": "string",
"description": "Path to user-defined local antiSMASH database.",
"fa_icon": "fas fa-layer-group",
"default": "None",
"help_text": "It is recommend to pre-download the antiSMASH databases to your machine and pass the path of it to this parameter, as this can take a long time to download - particularly when running lots of pipeline runs. \n\nSee the pipeline [documentation](https://nf-co.re/funcscan/usage#antismash) for details on how to download this. If running with docker or singularity, please also check `--bgc_antismash_installationdirectory` for important information."
},
"bgc_antismash_installationdirectory": {
"type": "string",
"description": "Path to user-defined local antiSMASH directory. Only required when running with docker/singularity.",
"fa_icon": "far fa-folder-open",
"default": "None",
"help_text": "This is required when running with **docker and singularity** (not required for conda), due to attempted 'modifications' of files during database checks in the installation directory, something that cannot be done in immutable docker/singularity containers.\n\nTherefore, a local installation directory needs to be mounted (including all modified files from the downloading step) to the container as a workaround."
},
"bgc_antismash_sampleminlength": {
"type": "integer",
"default": 1000,
"description": "Minimum longest-contig length a sample must have to be screened with antiSMASH.",
"fa_icon": "fas fa-ruler-horizontal",
"help_text": "This specifies the minimum length that the longest contig must have for the entire sample to be screened by antiSMASH.\n\nAny samples that do not reach this length will be not be sent to antiSMASH, therefore you will not receive output for these samples in your `--outdir`.\n\n> \u26a0\ufe0f This is not the same as `--bgc_antismash_contigminlength`, which specifies to only analyse contigs above that threshold but _within_ a sample that has already passed `--bgc_antismash_sampleminlength` sample filter!"
"help_text": "This specifies the minimum length that the longest contig must have for the entire sample to be screened by antiSMASH.\n\nAny samples that do not reach this length will be not be sent to antiSMASH, therefore you will not receive output for these samples in your `--outdir`.\n\n> ⚠️ This is not the same as `--bgc_antismash_contigminlength`, which specifies to only analyse contigs above that threshold but _within_ a sample that has already passed `--bgc_antismash_sampleminlength` sample filter!"
},
"bgc_antismash_contigminlength": {
"type": "integer",
Expand Down Expand Up @@ -942,7 +934,6 @@
},
"bgc_deepbgc_database": {
"type": "string",
"default": "None",
"fa_icon": "fas fa-layer-group",
"description": "Path to local deepBGC database folder."
},
Expand Down Expand Up @@ -1071,7 +1062,6 @@
},
"bgc_hmmsearch_models": {
"type": "string",
"default": "None",
"description": "Specify path to the BGC hmm model file(s) to search against. Must have quotes if wildcard used.",
"help_text": "HMMSearch performs biosequence analysis using profile hidden Markov Models.\nThe models are specified in`.hmm` files that are specified with this parameter\n\ne.g. \n\n```\n--bgc_hmmsearch_models '/<path>/<to>/<models>/*.hmm'\n```\n\nYou must wrap the path in quotes if you use a wildcard, to ensure Nextflow expansion _not_ bash!\n\nFor more information check HMMER [documentation](http://hmmer.org/).",
"fa_icon": "fas fa-layer-group"
Expand Down Expand Up @@ -1250,14 +1240,12 @@
"type": "boolean",
"description": "Display help text.",
"fa_icon": "fas fa-question-circle",
"default": false,
"hidden": true
},
"version": {
"type": "boolean",
"description": "Display version and exit.",
"fa_icon": "fas fa-question-circle",
"default": false,
"hidden": true
},
"publish_dir_mode": {
Expand All @@ -1281,7 +1269,6 @@
"type": "boolean",
"description": "Send plain-text email instead of HTML.",
"fa_icon": "fas fa-remove-format",
"default": false,
"hidden": true
},
"max_multiqc_email_size": {
Expand All @@ -1296,7 +1283,6 @@
"type": "boolean",
"description": "Do not use coloured log outputs.",
"fa_icon": "fas fa-palette",
"default": false,
"hidden": true
},
"hook_url": {
Expand Down Expand Up @@ -1335,23 +1321,20 @@
"type": "boolean",
"fa_icon": "far fa-eye-slash",
"description": "Show all params when using `--help`",
"default": false,
"hidden": true,
"help_text": "By default, parameters set as _hidden_ in the schema are not shown on the command line when a user runs with `--help`. Specifying this option will tell the pipeline to show all parameters."
},
"validationFailUnrecognisedParams": {
"type": "boolean",
"fa_icon": "far fa-check-circle",
"description": "Validation of parameters fails when an unrecognised parameter is found.",
"default": false,
"hidden": true,
"help_text": "By default, when an unrecognised parameter is found, it returns a warinig."
},
"validationLenientMode": {
"type": "boolean",
"fa_icon": "far fa-check-circle",
"description": "Validation of parameters in lenient more.",
"default": false,
"hidden": true,
"help_text": "Allows string values that are parseable as numbers or booleans. For further information see [JSONSchema docs](https://github.com/everit-org/json-schema#lenient-mode)."
}
Expand Down