NOEL-MNI / deepFCD

Automated Detection of Focal Cortical Dysplasia using Deep Learning
https://noel.bic.mni.mcgill.ca/projects/
BSD 3-Clause "New" or "Revised" License
10 stars 3 forks source link

Installation broken since packages not available anymore and tensorflow version not specified #16

Closed 1-w closed 1 year ago

1-w commented 1 year ago

Installation broken since packages not available anymore and tensorflow version not specified

Description

antspyx==0.3.5 not available

Expected Behavior

Actual Behavior

Possible Fix

works with the following combination:

antspyx==0.3.8 --only-binary=antspyx git+https://github.com/ravnoor/atlasreader@master#egg=atlasreader Theano==1.0.4 keras==2.2.4 h5py==2.10.0 matplotlib==3.5.1 mo-dots==9.147.22086 nibabel==3.2.2 nilearn==0.9.1 numpy==1.18.5 pandas==1.3.5 psutil==5.9.2 scikit-image==0.19.2 scikit-learn==1.0.2 scipy==1.7.3 setproctitle==1.2.3 tabulate==0.9.0 tqdm==4.64.0 xlrd==2.0.1 tensorflow==1.15.5 tensorflow_probability==0.8

Steps to Reproduce

1. 2. 3. 4.

Context

Your Environment

1-w commented 1 year ago

Which tensor flow version do you use?

ravnoor commented 1 year ago

TensorFlow isn't a hard dependency of the deepFCD project. It still runs on the deprecated Theano backend of Keras.

TensorFlow and TensorFlow-Probability are required by the ANTsPyNet dependency which is used for brain extraction. It is embedded within the deepMask project and is triggered only when preprocessing is done on the CPU (os.environ.get("BRAIN_MASKING") == "cpu"), since deepMask is resource-efficient only when run on the GPU.

I'll update deepMask with the following code so that modules aren't unnecessarily imported:

if os.environ.get("BRAIN_MASKING") == "cpu":
    from antspynet.utilities import brain_extraction