search

Ensembl / ensembl-vep

The Ensembl Variant Effect Predictor predicts the functional effects of genomic variants
https://www.ensembl.org/vep
Apache License 2.0
437 stars 150 forks source link

VEP 88: Update Dockerfile to build VEP 88.14 #1682

Open nuno-agostinho opened 1 month ago

nuno-agostinho commented 1 month ago

The current Dockerfile for 88.14 contains VEP 90. This is probably because the script to build the Docker image was run after releasing 90, given that the Dockerfile simply retrieves the latest default branch from the repo.

This PR intends to improve the situation by using instructions that more recent VEP Docker builds use, including the possibility of choosing the VEP version to install.

Testing

Locally build the image and run VEP with some options to see if it works.

likhitha-surapaneni commented 1 week ago

Hi @nuno-agostinho , just a small comment as the travis tests are currently failing. We may have to update the git branch names for travis build: https://github.com/nuno-agostinho/ensembl-vep/blob/e46dcbe28b9dfccc48d29ca3d21c8943af053d59/.travis.yml#L31-L35

nuno-agostinho commented 1 week ago

Hi @likhitha-surapaneni, thanks! Updated the Travis file with the correct branch, let's see.