search

ndaniel / fusioncatcher

Finder of Somatic Fusion Genes in RNA-seq data
GNU General Public License v3.0
142 stars 67 forks source link

Make Database Installation optional for DockerContainers #95

Closed Selonka closed 6 years ago

Selonka commented 6 years ago

Hi,

at first, thank you for that nice tool and the comfortable installation script. But would it be possible to separate the database installation from the of the installation script? So the databases can be installed/downloaded outside the container to the host? This would make the building process much faster and the container much lighter. Maybe you can add another command flag to specify the location of the database directory when running fusioncatcher.

VEP does it the same way if you are familiar with it ;)

Thank you and Greeting,

Selonka

ndaniel commented 6 years ago

That is a good idea!

Until a more permanent fix will be done one could edit manually the docker file and remove the string --download from the last line in order to have the database outside of the container.

Selonka commented 6 years ago

Hi, thank you for your quick response, I committed a new dockerfile and also a bash-script (altered from the script from bootstrap) to install the Database-files to a chosen destination on the host machine.