Hi <@U03EY2LC5V3>! I would like to start working ...
# variantbenchmarking
s
Hi @Kübra Narcı! I would like to start working on the task Add rtgtools/cnveval #266. I noticed it has been assigned since January and there hasn’t been any recent activity. Would it be okay if I take this task?
k
Hello, I think it is assigned to @GeorgiaKes, you need her consent :)
gratitude thank you 1
g
Of course! Feel free to work on that!
gratitude thank you 1
k
perfect, than feel free to assign it yourself there, but most importantly here is the issue in nf-core/modules https://github.com/nf-core/modules/issues/10393 start working there
👀 1
s
Hi @Kübra Narcı! The module and subworkflow are implemented and working. However, I ran into an issue with test data: cnveval requires truth VCF records wit SVTYPE=DUP or DEL, but the existing truth file (cnv_benchmark_calls.vcf) uses SVTYPE=<CNV:GAIN> and <LOH>, which cnveval doesn't recognize As a workaround, I generated a small truth VCF (assets/test_cnveval_truth.vcf) with 10 DUP/DEL records derived from the existing test data, and the pipeline runs end-to-end successfully with it What would be the preferred approach here? 1. Keep the generated truth VCF locally in assets/ 2. Upload it to nf-core/test-datasets 3. Something else
Снимок экрана 2026-03-12 в 14.59.06.png
k
Hello Sevda, are you creating the module in variantbenchmarking? I think it is best to add the tool to nf-core/modules first as rtgtools/cnveval and then we can pull the tool to variantbenchmarking. inside nf-core/modules/tests there should be more test cases. we prefer not to add tests to assets. If you cannot find a test case in nf-core we can think about another solution 🙂
gratitude thank you 1
s
oh yes, I've done according this instruction ok, I will add it to nf-core/modules first🙂
k
🙂 yea sorry this instruction is for in general issues for the pipeline, we also like to have tools registred in nf-core/modules to enable common usage (if they met to the criterias..)
s
Ok, I found instruction for adding a new module to nf-core/modules Thank you for the help!
Hi @Kübra Narcı! I've added the module to nf-core/modules and all comments seem to be addressed. Now I'm waiting for the merge. In the meantime, I’d like to continue working on the integration in variantbenchmarking. What would be the best way to proceed with that before the module is merged? Should I wait for the merge first?
k
Hello! thank you very much for the module. you can try to add the tool under subworkflows/local/sv_germline_benchmark/main.nf. This suppose to work
if (params.method.contains('rtgtools') && params.variant_type != "copynumber")
please also add a test case to make sure everything works through
gratitude thank you 1
let me know if you have any issues or problems
👌 1