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

Invalid aminoacid change detected due to GTF file phase ignored #541

Open
znikasz opened this issue Mar 15, 2022 · 2 comments
Open

Invalid aminoacid change detected due to GTF file phase ignored #541

znikasz opened this issue Mar 15, 2022 · 2 comments
Labels
Milestone

Comments

@znikasz
Copy link

znikasz commented Mar 15, 2022

Describe the bug
Using hg38_ensembl.ser, version 95 jannovar annotates change

chr1 939278 C > T

in transcript ENST00000455979 as p.(R2C). This is incorrect, the correct change is A1V.

In my opinion Jannovar ignores the frame shift (phase) data in GTF file, which is 2 in the case of ENST00000455979:

1       havana  CDS     939275  939460  .       +       2       gene_id "ENSG00000187634"; gene_version "11"; transcript_id "ENST00000455979"; transcript_version "1"; exon_number "1"; gene_name "SAMD11"; gene_source "ensembl_havana"; gene_biotype "protein_coding"; transcript_name "SAMD11-205"; transcript_source "havana"; transcript_biotype "protein_coding"; protein_id "ENSP00000412228"; protein_version "1"; tag "cds_start_NF"; tag "mRNA_start_NF"; transcript_support_level "2";

To Reproduce
Steps to reproduce the behavior:

  1. Annotate chr1 939278 C > T and check it's protein change in ENST00000455979 .

Expected behavior
Aminoacid change should be A1V

Additional context
Please check if you don't ignore phase in CDS entries.

@znikasz znikasz added the bug label Mar 15, 2022
@znikasz znikasz changed the title Invalid aminoacid change detected due to GTF file frame ignored Invalid aminoacid change detected due to GTF file phase ignored Mar 15, 2022
@holtgrewe holtgrewe added this to the v0.39 milestone Mar 29, 2022
@holtgrewe
Copy link
Member

Confirmed, phase is indeed ignored.

@holtgrewe
Copy link
Member

We need to ask ENSEMBL of the meaning of frame information for CDS on reverse strand for proper implementation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

2 participants