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

Inversion mapping is not palindrome aware #285

Closed
reece opened this issue Nov 6, 2015 · 1 comment
Closed

Inversion mapping is not palindrome aware #285

reece opened this issue Nov 6, 2015 · 1 comment
Labels
bug Something isn't working
Milestone

Comments

@reece
Copy link
Member

reece commented Nov 6, 2015

Originally reported by: Peter Causey-Freeman (Bitbucket: PeteCauseyFreeman, GitHub: Unknown)


Hi All,

I'm reporting this as a bug but nothing is actually broken.

It may be a fussy pedantic thing, but the mapper for hgvs is not palindrome aware when mapping inversions. For example, I queried both hgvs and mutalyzer with the variant description NM_000088.3:c.589_600inv into both mutalyzer and hgvs.

hgvs returns (workflow evm.c_to_g then evm.g_to_c to collect sequence data) NM_000088.3:c.589_600invGGTCCCCAAGGC

mutalyzer, however returns the following message (NOTE, mutalyzer reports the :n. coordinates not the input :c. coordinates in its warning message)

"Sequence "GGTCCCCAAGGC" at position 715_726 is a partial palindrome (the first 1 nucleotide(s) are the reverse complement of the last one(s)), the HGVS notation prescribes that it should be "GTCCCCAAGG" at position 716_725."

Although the inversion generated by hgvs was not technically incorrect and the protein consequence is the same as reported by mutalyzer (I have written some code to generate protein consequences from hgvs nucleotide inversion objects), HGVS nomenclature would dictate that the correct output would be NM_000088.3:c.590_599invGTCCCCAAGG not NM_000088.3:c.589_600invGGTCCCCAAGGC.

Regards

Pete


@reece
Copy link
Member Author

reece commented Jul 28, 2016

Original comment by Reece Hart (Bitbucket: reece, GitHub: reece):


fixes #285, #334, #335, #324, #340: inversions parsing, formatting, and normalization

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant