-
Notifications
You must be signed in to change notification settings - Fork 0
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
116 - Update CUDA in Dockerfile and bump requirement versions #117
Conversation
This comment was marked as off-topic.
This comment was marked as off-topic.
Would be great to see the actual source of the error which seems to be cut off at the top :) |
Sorry for the bad screenshot, I will add a better one tomorrow. |
@samuelkuehnel Thanks. I changed the installation method (which is also faster and simpler) for CUDA. Maybe that helps. Also removed libgit as it was a workaround for DVC and it seems to be fine without it now. |
The error seems to be on the PC as the git action that builds the image completes without an error. I still get an error when running @JuliusMiller also tried to run |
This comment was marked as off-topic.
This comment was marked as off-topic.
Thanks for testing it. Hmm maybe the lab pcs need an |
Found a possible workaround for the error. Can you or @JuliusMiller try again? Thanks! |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
Did some more research, as there is no helpful error message anymore. |
This comment was marked as off-topic.
This comment was marked as off-topic.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update works. Image is built and container starts as expected
Description
Update CUDA in Dockerfile and bump requirement versions. See issue below for reasons why.
Fixes #116
Type of change
Please delete options that are not relevant.
Does this PR introduce a breaking change?
No
Checklist: