Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Assign SNPs on chromosome 0 #13

Open
apriha opened this issue Jun 14, 2019 · 2 comments
Open

Assign SNPs on chromosome 0 #13

apriha opened this issue Jun 14, 2019 · 2 comments
Labels
enhancement New feature or request

Comments

@apriha
Copy link
Owner

apriha commented Jun 14, 2019

Some newer Family Tree DNA famfinder files seem to have SNPs assigned to chromosome 0. Similar to SNPs._assign_par_snps, use the RefSNP API to assign each of these SNPs to a chromosome and update the SNP's position to be consistent with the assembly.

@willgdjones
Copy link
Contributor

I have noticed that for AncestryDNA files, the _assign_par_snps step can take a substantial amount of time (> 1 minute), because of repeated API requests. I have currently bypassed this issue by setting assign_par_snps=False. Is there a way we can speed this up with the VCF_endpoint like you suggest in #19.

@apriha
Copy link
Owner Author

apriha commented Dec 1, 2019

Yeah, I agree that having a resource like discussed in #19 to lookup SNPs would help here as well and would reduce repeated API calls. I'll start looking into this as part of #42.

@apriha apriha added the enhancement New feature or request label Dec 20, 2019
willgdjones pushed a commit that referenced this issue Jul 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants