Skip to content
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

Fixed BaseImage Issue for Nvidia Cuda not being found #1182

Closed
wants to merge 3 commits into from

Conversation

TeenPhilosopher
Copy link

Fixed the format of the baseImage urls. Changed cudnn8-devel to base.

TEST

Signed-off-by: RenoirLover <[email protected]>
return


Signed-off-by: RenoirLover <[email protected]>
Replace cudnn8-devel with base. This fixes the error where it can't find baseImages and prevents building the docker.

Signed-off-by: RenoirLover <[email protected]>
@mattt
Copy link
Contributor

mattt commented Jul 7, 2023

Hi, @TeenPhilosopher. Thanks for your interest in contributing to Cog. The issue with base images not being found (#1162) has to do several versions being untagged from Docker Hub, which I believe is the case for all variants, including base. And despite their "BaseImage" naming, we actually do want the devel images, which include header files and development tools that models rely on.

@mattt mattt closed this Jul 7, 2023
@TeenPhilosopher
Copy link
Author

Okay but the matter of fact is that it’s not working as is. I tried it with this version of the source code and it built correctly and the model ran… I want to push this model to replicate (and indeed I did but it wouldn’t accept it as valid because cog had been modified by me) but I can’t as it stands. Any thoughts?

@mattt
Copy link
Contributor

mattt commented Jul 7, 2023

@TeenPhilosopher What specifically wasn't working for you? Could you share the commands you tried and their output?

As I mentioned before, there was a recent issue with CUDA base images (#1162) that's now fixed by v0.8.0. Have you tried updating to the latest version of Cog?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants