-
Notifications
You must be signed in to change notification settings - Fork 22
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
example/llm
: Re-evaluate open-webui environment variables
#230
Comments
We needn’t have to assign Also, this ENV doesn’t affect |
|
|
{
RAG_EMBEDDING_ENGINE = "ollama";
RAG_EMBEDDING_MODEL = "mxbai-embed-large:latest";
} should be fine, since otherwise Open WebUI will use sentence-transformers to fetch the embedding models, which would require |
This, verbatim, sounds like a good candidate for a comment on top of these env vars. The rest can be either commented out or removed. |
For details, see: - #230 (comment) - #230 (comment) - #230 (comment) - #230 (comment) Additionally introduced `WEBUI_SECRET_KEY`, see why: #229 (comment)
For details, see: - #230 (comment) - #230 (comment) - #230 (comment) - #230 (comment) Additionally introduced `WEBUI_SECRET_KEY`, see why: #229 (comment)
services-flake/example/llm/flake.nix
Lines 48 to 58 in f360c8f
Keep only environment variables (which were introduced in #227) that are strictly necessary, while leaving the rest commented out.
Consider the implications of
DEVICE_TYPE = "cpu";
especially when GPU is enabled.Our examples should a) "just work", be b) simple and minimal, c) well-documented (liberal use of comments, for example).
The text was updated successfully, but these errors were encountered: