Skip to content

Fix VCF position int overflow (long long) & fix args.ploidy typo (arggs -> args)#47

Open
wnsplim wants to merge 1 commit into
popgenmethods:mainfrom
wnsplim:fix-vcf-pos-overflow
Open

Fix VCF position int overflow (long long) & fix args.ploidy typo (arggs -> args)#47
wnsplim wants to merge 1 commit into
popgenmethods:mainfrom
wnsplim:fix-vcf-pos-overflow

Conversation

@wnsplim

@wnsplim wnsplim commented May 29, 2026

Copy link
Copy Markdown

Two small fixes:

  1. VCF position integer overflow: Sampler.cpp reads the VCF POS field into an int, which results in overflow with (simulated) VCFs with a length longer than ~2.1Gb. Solution: simply swap int to long long
  2. Typo in singer_master: arggs.ploidy -> args.ploidy.

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.

1 participant