Skip to content

MNVs - other variants considered as part of phased MNV annotation #624

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
Open
julie-sullivan wants to merge 2 commits into
base: release-5.2.x
Choose a base branch
from
Open

MNVs - other variants considered as part of phased MNV annotation #624

julie-sullivan wants to merge 2 commits into from

Conversation

@julie-sullivan
Copy link
Collaborator

@julie-sullivan julie-sullivan commented Nov 3, 2022 •
edited
Loading

MNVs - other variants considered as part of phased MNV annotation


// variant additionalAttributes
"additionalAttributes": {
    "phasedTranscripts": {
        "attribute": {             
            "ENST0034245234": "1:343423:C:G,1:343423:CA:G",
            "ENST0034245236": "1:343423:C:G,1:343423:CA:G",   
            "ENST0034245237": "1:343423:C:G",
        }
    }
}
  • added phased transcripts + variants to additional attributes
  • added passing unit test
  • Some queries were missing the data release variable - I've added. Please double check!

Please see the PR for the changes in v4: #586

@julie-sullivan julie-sullivan self-assigned this Nov 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jtarraga jtarraga Awaiting requested review from jtarraga

At least 1 approving review is required to merge this pull request.

Assignees

@julie-sullivan julie-sullivan

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@julie-sullivan