Give decent error message if model not found #2
Conversation
| curl -fsSL https://raw.githubusercontent.com/containers/ramalama/main/install.sh | sudo bash | ||
|
|
||
| set +o pipefail | ||
| ./ramalama -h | grep Usage: | ||
| set -o pipefail | ||
|
|
||
| ramalama pull granite-code |
| @@ -11,14 +11,14 @@ main() { | |||
| apt install -y python3-autopep8 | |||
| fi | |||
|
|
|||
| ./podman-build.sh | |||
There was a problem hiding this comment.
The thing is about this is, it doesn't build ramalama, it only builds container images...
Maybe container-build.sh ?
Not a blocking issue for this PR anyway
There was a problem hiding this comment.
We can catch in a follow on PR if needs be
| @@ -7,18 +7,22 @@ available() { | |||
| main() { | |||
| set -ex -o pipefail | |||
|
|
|||
| if ! available autopep8 && available apt; then | |||
| apt install -y python3-autopep8 | |||
| if ! available autopep8; then | |||
There was a problem hiding this comment.
I already fixed the sudo thing, it's one of the reasons we have conflicts in this PR
|
@ericcurtin can we use the small model for ci rather then granite? |
|
@rhatdan I'll switch to moondream from Ollama registry, it's smaller, although there may be even smaller than that |
|
We just need a chmod a+x or whatever to fix CI here I think, assuming python style is autopep8 auto-formatted, etc. |
|
We can do the moondream change is this PR if needs be too |
|
"tinyllama" is even smaller |
Rename podman_build.sh to build.sh Signed-off-by: Daniel J Walsh <[email protected]>
Rename podman_build.sh to build.sh