-
Notifications
You must be signed in to change notification settings - Fork 39
Integrate Hackathon vrs-in-a-box into vrs-python #555
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
Conversation
|
@larrybabb and @theferrit32. As discussed at the think tank, I have started the work to integrate vrs-in-a-box into vrs-python. I think there are a still a few outstanding items. My idea here is that I am throwing this over the fence to you to wrap up. |
|
@theferrit32 Done. Tests are failing in Python 3.13 for some unrelated reason. |
larrybabb
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
per think tank review - approved
|
|
||
| # Either 'GRCh38' or 'GRCh37' | ||
| ARG ASSEMBLY="GRCh38" | ||
| ARG VRS_PYTHON_VERSION="2.1.2" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| ARG VRS_PYTHON_VERSION="2.1.2" | |
| ARG VRS_PYTHON_VERSION="2.2.0" |
| # GRCh38 | ||
| docker build --build-arg ASSEMBLY=GRCh38 --build-arg VRS_PYTHON_VERSION=2.1.2 -t ga4gh/vrs-vcf-annotator-grch38:2.1.2 . | ||
| # GRCh37 | ||
| docker build --build-arg ASSEMBLY=GRCh37 --build-arg VRS_PYTHON_VERSION=2.1.2 -t ga4gh/vrs-vcf-annotator-grch37:2.1.2 . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
version issue here?
Moved code from the Hackathon vrs-in-a-box project over in to a subdirectory within vrs-python.
The code includes full instructions for how to create a vrs-in-a-box Docker image from scratch
and examples for how to run using Docker.
The following items are (probably) still needed to bring this work to completion: