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.

Sign up for GitHub

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

Jump to bottom

reev does not load any boxes for a specific variant #447

Closed
sczakihl opened this issue Feb 6, 2024 · 2 comments · Fixed by #450 or bihealth/reev-frontend-lib#98
Closed

reev does not load any boxes for a specific variant #447

sczakihl opened this issue Feb 6, 2024 · 2 comments · Fixed by #450 or bihealth/reev-frontend-lib#98
Labels
bug Something isn't working
Milestone
0.6.0

Comments

@sczakihl
Copy link
Collaborator

sczakihl commented Feb 6, 2024

Describe the bug
For this indel nothing is loading: chrX:102884958:T:TTCGGAGGAGCAGTCC also nothing when entering as NM_004780.3(TCEAL1):c.124_138dup

Other indels (e.g. chr4:113568635:ATCAAAAGTAAAGAAAATTAT:A) work however.

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'reev'
  2. enter TCEAL1 variant above
  3. Scroll down to 'anywhere'
  4. See error

Expected behavior
results page should load.

Screenshots
grafik
@sczakihl sczakihl added the bug Something isn't working label Feb 6, 2024
@holtgrewe holtgrewe added this to the 0.6.0 milestone Feb 6, 2024
@holtgrewe
Copy link
Member

Root Cause Analysis
The seqvarInfo store made the wrong assumption that there must be HPO terms for each gene.

holtgrewe added a commit to bihealth/reev-frontend-lib that referenced this issue Feb 6, 2024
@holtgrewe
fix: remove requirement of genes having HPO terms (bihealth/reev#447)
cbed60a
holtgrewe added a commit to bihealth/reev-frontend-lib that referenced this issue Feb 6, 2024
@holtgrewe
fix: remove requirement of genes having HPO terms (bihealth/reev#447) (…
d570f1c 
…#98)
holtgrewe added a commit that referenced this issue Feb 6, 2024
@holtgrewe
fix: bump rfl to remove issue with sequenceInfo store (#447)
fa2d6aa
This was linked to pull requests Feb 6, 2024
fix: bump rfl to remove issue with sequenceInfo store (#447) #450
Merged
fix: remove requirement of genes having HPO terms (bihealth/reev#447) bihealth/reev-frontend-lib#98
Merged
holtgrewe added a commit that referenced this issue Feb 6, 2024
@holtgrewe
fix: bump rfl to remove issue with sequenceInfo store (#447) (#450)
1d15de5
@holtgrewe
Copy link
Member

@sczakihl please confirm that this works now

@varfish-bot varfish-bot mentioned this issue Feb 6, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
0.6.0
2 participants
@holtgrewe @sczakihl