-
Notifications
You must be signed in to change notification settings - Fork 78
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
make settings.yaml persistent!!! #40
Comments
TLDR; Hi, thanks for sharing your issue - and big thanks for submitting a PR too! You prompted me to look into this a bit... normally Docker images are stateless - in this case, that means the settings should be configured via During the course of examining the upstream code, I noted that there is an option to specify a settings file using the I do not have an exact implementation for this feature yet, but two approaches come to mind: 1) symlink the default file to a mountable folder and add business logic to the entrypoint script to handle its initialisation, or 2) specify a custom file using the setting flag I mentioned earlier (although I do not 100% like this one because it relies on passing an argument to the program, which a user may override). For an immediate fix, I suggest just mounting a |
found this "--settings $FILE" too but this dosnt work update: nevermind i got it working |
find a way to make it persistent. e.g. apply a patch after u get https://github.com/oobabooga/text-generation-webui.git in dockerfile to change the default dir for settings.yaml to eg /extensions
it's absolutely annoying
heres the patch u have to use:
https://github.com/Gee1111/text-generation-webui/blob/main/make_settings_persistent.patch
edit send u a merge request but its better to put the make_settings_persistent.patch in your own repo and access it from there
The text was updated successfully, but these errors were encountered: