-
Notifications
You must be signed in to change notification settings - Fork 45
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
Fix Dockerfiles and add GitHub Actions job #252
Conversation
DeepCSR
modelDeepCSR
model
@kabilar I just looked at this randomly and I wonder what has it to do with the missing DeepCSR model. That is part of the nobrainer-zoo or trained-models repos. |
Hi @hvgazula, I was attempting to build the Docker images for this repository, but received an error during the build when the models were being downloaded from the |
Hello @kabilar I tested the changes you proposed on my local Windows machine. The Did you test the changes with the GPU dockerfile as well? If so, did it work for you and how did you build it? |
Hi @gaiborjosue, good catch. I did not test the build of the The commit directly above fixes this issue, by changing the base images and the |
DeepCSR
modelDeepCSR
model and tensorflow-gpu
version
DeepCSR
model and tensorflow-gpu
version
no need to generate CHANGELOG - that's automatically generated. |
Hi @satra, thank you for clarifying. I have removed the updates to the CHANGELOG. |
Types of changes
Summary
When building the Docker images, I encountered an error with downloading the
DeepCSR
model.Command
docker build -t neuronets/nobrainer:master-cpu -f docker/cpu.Dockerfile .
Error
Click to expand the full error stack
When building the GPU Docker image, encountered an error since
tensorflow-gpu
is deprecated.Command
docker build -t neuronets/nobrainer:master-gpu -f docker/gpu.Dockerfile .
Error
ERROR: Failed building wheel for tensorflow-gpu
Click to expand the full error stack
Changes
osf-storage
DataLad data source.tensorflow
package and version.Checklist
nobrainer
andtensorflow
Acknowledgment