Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
I'm not a coder so these changes definitely need to be reviewed before being accepted!
--build-args
statement but the following lines needed the values in those args as well. So I replaced that line with awhile
that builds the--build-args
statement and assigns the vals to the vars.invokeai-nvidia
toinvokeai-cuda
. Despite the comment in the.env
file saying that the values should be "nvidia" or "rocm", "nvidia" doesn't work but "cuda" does. This line may be the reason that "cuda" is required. And since$GPU_DRIVER
must be set to "cuda" for an Nvidia GPU,$profile
is set based on$GPU_DRIVER
, and$service_name
is set based on$profile
,$service_name
ends up beinginvokeai-cuda
(notinvokeai-nvidia
).compose.override.yaml
torun.sh
in case a user has such a need. I use Traefik as proxy and middleware plugins ipwhitelist and traefik-forward-auth, so I usecompose.override.yaml
so that I don't have to modifydocker-compose.yaml
and thus have git telling me I need to commit a change.Related Issues / Discussions
NA
QA Instructions
NA
Merge Plan
NA
Checklist