Skip to content

Add HG002 ONT SV genotyping test data - #2302

Merged
aksenia merged 1 commit into
nf-core:modulesfrom
aksenia:sniffles-genotypevcf
Oct 8, 2026
Merged

aksenia merged 1 commit into
nf-core:modulesfrom
aksenia:sniffles-genotypevcf

Conversation

@aksenia

@aksenia aksenia commented Oct 7, 2026 •

Copy link
Copy Markdown

Adds a small HG002 ONT subset (BAM 0.77 MB + index) and a matching VCF of 15 known structural variants (INS, DEL, INV, BND) for testing SV genotyping tools that genotype a given SV set, such as Sniffles --genotype-vcf. They are needed by the upcoming sniffles/genotypevcf module in nf-core/modules.

The BAM covers four small windows of GRCh38, subsampled to ~12x by read name, with base qualities, MM/ML/mv/ts/ns/HP/PS tags, @PG lines and UR paths removed. The README in the folder documents provenance, files and the expected genotypes (1 0/0, 7 0/1, 7 1/1 with Sniffles 2.8.1).

Files: data/genomics/homo_sapiens/nanopore/bam/HG002_ont_sv_genotype/

No existing long-read BAM in test-datasets has a matching known-SV VCF at a depth where heterozygous and homozygous genotypes can be called. The SAVANA COLO829 subset (~1-2x, no SV set) and the generic nanopore/bam/test.sorted.bam (contigs < 1 Mb, which Sniffles skips) were considered.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would it be possible to make it <1MB or even smaller? the bigger the test-dataset the longer tests take (and the more bloated the repo gets) 🙂

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for having a look @mashehu ! Yes I can reduce number of regions and re-sample. I actually looked for nf-core modules guideline about the advised size limits and did not find any. Did I miss it? Patch incoming.

@mashehu mashehu Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we are rather vague by saying "as small as possible". but you can look at other data sets where we for example often just subsample to chr22

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

re-uploaded smaller dataset @mashehu.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please drop the old file from the previous commit and force push, to keep the git history clean 🙂

@aksenia aksenia Oct 8, 2026 •

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK done @mashehu :)

@aksenia
aksenia force-pushed the sniffles-genotypevcf branch from 04752da to 9a67221 Compare October 8, 2026 10:50

@mashehu mashehu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thanks!

@aksenia
aksenia merged commit 2577e91 into nf-core:modules Oct 8, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants