Filter operator fix

[jmattock5]

Fix for the filter ratio threshold logic. Documentation stated that samples where less than 10% of reads were kept after filtering were flagged and not assembled. However the code was flagging samples that met this criteria:

(af_total_reads / bf_total_reads) <= params.filter_ratio_threshold

It was discarding samples where the proportion of passed reads was less than 90%. I've changed the filter threshold in the config to 10% to flag only the samples with less than 10% of the reads retained. I've also updated the description in the workflow and the README.

We haven't yet modified the tests affected by this change.

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
• Make sure your code lints (nf-core 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>).
• README.md is updated (including new tool citations and authors/contributors).

[Vkale1]

This works :) thanks

[mberacochea]

This is great, thank you for spotting and fixing the issue @jmattock5.

There is one extra file that needs to be modified -> https://github.com/EBI-Metagenomics/miassembler/blob/main/nextflow_schema.json#L165. This schema json file is used by the nf-schema plugin, which is sort of a argparse for nextflow

[Ge94]

Fab! Thank you Jenny :)

[mberacochea]

Super, thanks @jmattock5