bioconductor_docker icon indicating copy to clipboard operation
bioconductor_docker copied to clipboard

Pip install torch

Open jwokaty opened this issue 2 years ago • 3 comments

We pip installed torch for the package Bioc ASL. See https://github.com/Bioconductor/BBS/pull/143. You should install this in the docker if it's not already installed.

jwokaty avatar May 03 '22 20:05 jwokaty

torch is 750Mb. Is this really that important for one package? I'm having second thoughts because that is the same reason we don't install TensorFlow.

Downloading torch-1.11.0-cp38-cp38-manylinux1_x86_64.whl (750.6 MB)

nturaga avatar May 25 '22 15:05 nturaga

This is for the package APL. If you don't want to include it in the docker image because of the size, I understand and this issue can be closed, but I wanted to document this dependency for bioconductor docker.

jwokaty avatar May 25 '22 16:05 jwokaty

Thanks for documenting this @jwokaty 👍🏽 I appreciate it.

nturaga avatar May 25 '22 16:05 nturaga