eager icon indicating copy to clipboard operation
eager copied to clipboard

DSL2: Add MultiVCFAnalyzer

Open aidaanva opened this issue 1 year ago • 2 comments

Adding MultiVCFAnalyzer

aidaanva avatar May 17 '24 08:05 aidaanva

Note: when adding multivcf analyzer ploidy MUST be set at 2 for unified genotyper.

Current default behavior is that any fasta without explicit ploidy will be set at ploidy 2 for the UG execution.

Please add a check to correctly check that ploidy is not erroneously set at something other than 2 (easiest would be to just add check for params.run_multivcfanalyzer to then force ploidy 2 for UG execution in modules config..., but maybe better to add explicit WARN statement at runtime that ploidy will be changed so that users know what to expect 🤷 ).

ilight1542 avatar Mar 26 '25 12:03 ilight1542

Good point @ilight1542! I will add the necessary checks/enforce the behaviour

aidaanva avatar Mar 26 '25 12:03 aidaanva