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

Add VAT clinvar reverse complement #9070

Open
wants to merge 7 commits into
base: ah_var_store
Choose a base branch
from

Conversation

RoriCremer
Copy link
Contributor

@RoriCremer RoriCremer commented Jan 7, 2025

# Note that inside the clinvar array, are multiple objects that may or may not be the one we are looking for. We check by making sure the ref and alt are the same
# Note that inside the clinvar array, are multiple objects that may or may not be the one we are looking for.
# We check by making sure the ref and alt are the same including any reverse complements
## TODO add clinvar star rating!!!
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you going to do this work in this ticket or is this another ticket?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

another ticket

Comment on lines +248 to +249
## TODO add the ("variationId") and the ("reviewStatus")--note that the reviewStatus will need to maintain the ordering of the significance arrays
## we need to do this with a tuple so that the reviewStatus lines up with the significance (since significance seems to be an array, while star is a single value)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also wondering if this work is for this ticket / PR or another

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes another ticket--but I'm in here, it seemed like a waste to not note where it needed to live

@RoriCremer RoriCremer marked this pull request as ready for review January 22, 2025 00:11
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.

2 participants