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

Update run.sh #166

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update run.sh #166

wants to merge 1 commit into from

Conversation

true1ann
Copy link

@true1ann true1ann commented Nov 2, 2024

The issue: original used docker compose instead of docker-compose, which lead to issues (docker compose does NOT support -f flag.)

other details:

BEFORE:

❯ sudo ./run.sh --model 7b --with-cuda
unknown shorthand flag: 'f' in -f
See 'docker --help'.

Usage:  docker [OPTIONS] COMMAND

A self-sufficient runtime for containers

Common Commands:
  run         Create and run a new container from an image
  exec        Execute a command in a running container
  ps          List containers
  ...

then i analyzed how run.sh was made, and added the change

AFTER:

❯ sudo ./run.sh --model 7b --with-cuda
WARN[0000] /home/<hidden>/projects/llamagpt/llama-gpt/docker-compose-cuda-ggml.yml: the attribute `version` is obsolete, it will be ignored, please remove it to avoid potential confusion
[+] Building 30.2s (4/28)
...

(i just made an install so it started compiling)

The issue: original used `docker compose` instead of `docker-compose`, which lead to issues (`docker compose` does NOT support -f flag.)
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.

1 participant