-
Notifications
You must be signed in to change notification settings - Fork 94
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
implement support for repeats #113
Comments
@reece would it be reasonable to break this issue up into a separate ticket for each of the above "gotchas" bullet points and tackle them separately or some logical order that would allow us to get the first bullet resolved and then move on to a refactoring to handle the subsequent concerns? |
@andreasprlic This is the ticket you and I discussed this morning. I would love to do an MVP of repeat syntax support in the hgvs package if that is possible. @reece rightfully points out the nuances and gotchas involved with this syntax. I'm looking for the basics like supporting only the first bullet he points out. Here's the way I think we could/should deliver this feature in this module. First, the basics
Second, ranges
Again, while the parens () convey uncertainty there are many examples in the wild of not using the parenthesis on ranges. I'd like to make sure we can support these non-compliant representations since they are fairly prevalent. Here's a few examples from clinvar....
Third, complex repeats
Fourth, genotypes / hets
@andreasprlic Any assistance or management you can assist to break this up so we can start delivering on it would be wonderful. |
This issue is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 7 days. |
Did a small demo in this branch for how to use a fully justified representation of a variant to easily identify basic repeats: |
Originally reported by: Reece Hart (Bitbucket: reece, GitHub: reece)
See comments in hgvs.pymeta.
http://www.hgvs.org/mutnomen/recs-DNA.html#var
Lots of gotchas:
This definitely merits a feature branch.
Links
The text was updated successfully, but these errors were encountered: