epi2me-labs / wf-human-variation

Other
96 stars 42 forks source link

Calculate repeat expansion of RFC1 #135

Closed vroni closed 7 months ago

vroni commented 8 months ago

I have tried the wf-human-variation on a few of our samples and there is no calculation of the repeat expansion of the RFC1 locus. The other loci are calculated as expected. RFC1 does not show up in the str-report and does not appear in the wf_str.straglr.tsv output file. However, when looking at the code I can see that RFC1 is configured correctly in the wf_str_repeats.bed regions file as well as the variant_catalog_hg38.json annotation file. I was not able to find anything in the code that pointed to an exclusion of the region but I am wondering if there is a deliberate omission within the nextflow workflow of RFC1 that I am not seeing?

According to this GitHub Issue it looks like the straglr developers have already considered RFC1 and implemented for the locus.

vlshesketh commented 8 months ago

Hi @vroni thank you for reporting this. We are aware that there is an issue reporting the repeat expansion on RFC1 and will update with a fix shortly.

vroni commented 8 months ago

Perfect, thanks. I'm looking forward to the update and will try it when it is available.

vlshesketh commented 7 months ago

Hi @vroni, we have released a new version of the workflow, v1.11.0, containing an update to straglr which allows RFC1 repeats to be called. Please let us know if you experience any further issues.