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

SPDI normalization with refseq files #76

Open
wants to merge 28 commits into
base: main
Choose a base branch
from

Conversation

mihaitodor
Copy link
Collaborator

@mihaitodor mihaitodor commented Oct 5, 2023

Description

Follow-up to #69.

How Has This Been Tested?

  • Integration Tests
  • Unit Tests

@mihaitodor mihaitodor mentioned this pull request Oct 5, 2023
2 tasks
@mihaitodor mihaitodor force-pushed the mihaitodor-spdi-normalization-with-seq-files branch 2 times, most recently from 8feee51 to a4c1ffe Compare October 5, 2023 15:53
@mihaitodor mihaitodor force-pushed the mihaitodor-spdi-normalization-with-seq-files branch from a4c1ffe to e03580b Compare October 19, 2023 01:22
@mihaitodor mihaitodor force-pushed the mihaitodor-spdi-normalization-with-seq-files branch 7 times, most recently from eef7272 to e0d2405 Compare December 14, 2023 18:49
@mihaitodor mihaitodor force-pushed the mihaitodor-spdi-normalization-with-seq-files branch from 16f109b to bfa3861 Compare December 22, 2023 02:23
@mihaitodor mihaitodor force-pushed the mihaitodor-spdi-normalization-with-seq-files branch 5 times, most recently from 66d10f1 to e53861a Compare January 25, 2024 16:00
@mihaitodor mihaitodor force-pushed the mihaitodor-spdi-normalization-with-seq-files branch 2 times, most recently from 84546e7 to ebc6e5a Compare April 4, 2024 23:30
Need to run the Flask server in a background thread so the
seqfetcher endpoint is available for the hgvs library to call.
Details here: https://stackoverflow.com/questions/77676757/run-a-full-flask-server-under-pytest-to-handle-http-requests-on-a-given-port

Also need to pick the relevat transcript in `process_NC_HGVS()` in
such a way that we select the one with the most up to date version
so we don't end up unable to find the RefSeq data because it looks
like we don't have all the RNA transcripts that UTA knows about.
This is a workaround for some liftover failures: biocommons/hgvs#717
Also remove them from the variants
@mihaitodor mihaitodor force-pushed the mihaitodor-spdi-normalization-with-seq-files branch from ebc6e5a to 3a62166 Compare May 16, 2024 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant