[GH-ISSUE #231] Requirement file rework not reflected for docker #140

Closed
opened 2026-10-15 18:04:56 +00:00 by bsakaguchi · 5 comments
Owner

Originally created by @MNarath1 on GitHub (Mar 29, 2024).
Original GitHub issue: https://github.com/SillyTavern/SillyTavern-Extras/issues/231

Currently the docker compose and Dockerfile don't seem to be adapted for the change how requirement files are handled as an example the full install thats commented out is still with requirement-complete.txt even trough that file doesn't exist anymore and it doesn't at all even mention rvc nor its not dedicated requirement file.
Also pretty sure anything regarding rvc and other stuff are not part of any volume so won't survive a container rebuild

Originally created by @MNarath1 on GitHub (Mar 29, 2024). Original GitHub issue: https://github.com/SillyTavern/SillyTavern-Extras/issues/231 Currently the docker compose and Dockerfile don't seem to be adapted for the change how requirement files are handled as an example the full install thats commented out is still with requirement-complete.txt even trough that file doesn't exist anymore and it doesn't at all even mention rvc nor its not dedicated requirement file. Also pretty sure anything regarding rvc and other stuff are not part of any volume so won't survive a container rebuild
Author
Owner

@Cohee1207 commented on GitHub (Mar 30, 2024):

It's possible to specify multiple requirement files like this: https://github.com/SillyTavern/SillyTavern-Extras/blob/main/docker/docker-compose.yml#L21

Can't say anything about volumes, I don't have enough Docker expertise for that.

<!-- gh-comment-id:2028042934 --> @Cohee1207 commented on GitHub (Mar 30, 2024): It's possible to specify multiple requirement files like this: https://github.com/SillyTavern/SillyTavern-Extras/blob/main/docker/docker-compose.yml#L21 Can't say anything about volumes, I don't have enough Docker expertise for that.
Author
Owner

@MNarath1 commented on GitHub (Mar 30, 2024):

It's possible to specify multiple requirement files like this: https://github.com/SillyTavern/SillyTavern-Extras/blob/main/docker/docker-compose.yml#L21

Can't say anything about volumes, I don't have enough Docker expertise for that.

Yeah i figured that out thankfully i am currently working to get all the modules to work inside docker and add them as templates to the docker composer file(including the changes needed so that you can actually connect to the docker while in bridge mode) and am pretty much done RVC works now fine in it and i added the volumes only problem now is that for some reason it doesn't add the proper folders into the volumes for the models folder under data/models so i had to manually add those folders for the docker to work properly but after i get that done i will probably make a PR.
So basically my current docker setup survives trough rebuilds of the docker but it still doesn't properly sets itself up on first time container build.

<!-- gh-comment-id:2028442540 --> @MNarath1 commented on GitHub (Mar 30, 2024): > It's possible to specify multiple requirement files like this: https://github.com/SillyTavern/SillyTavern-Extras/blob/main/docker/docker-compose.yml#L21 > > Can't say anything about volumes, I don't have enough Docker expertise for that. Yeah i figured that out thankfully i am currently working to get all the modules to work inside docker and add them as templates to the docker composer file(including the changes needed so that you can actually connect to the docker while in bridge mode) and am pretty much done RVC works now fine in it and i added the volumes only problem now is that for some reason it doesn't add the proper folders into the volumes for the models folder under data/models so i had to manually add those folders for the docker to work properly but after i get that done i will probably make a PR. So basically my current docker setup survives trough rebuilds of the docker but it still doesn't properly sets itself up on first time container build.
Author
Owner

@MNarath1 commented on GitHub (Mar 30, 2024):

Without the proper folders the docker will just runtime since rvc does not seem to have a fallback if the folder inside data/models doesn't exist

<!-- gh-comment-id:2028443009 --> @MNarath1 commented on GitHub (Mar 30, 2024): Without the proper folders the docker will just runtime since rvc does not seem to have a fallback if the folder inside data/models doesn't exist
Author
Owner

@mrguymiah commented on GitHub (Apr 2, 2024):

It's worth noting that, even with the updated requirements-rvc.txt, I'm getting an INFO error requesting I install tensorboardX. Not sure if that's required to get the functionality up and going. Working on this all myself. I'd appreciate any info you can give on this.

image

<!-- gh-comment-id:2031026373 --> @mrguymiah commented on GitHub (Apr 2, 2024): It's worth noting that, even with the updated requirements-rvc.txt, I'm getting an INFO error requesting I install tensorboardX. Not sure if that's required to get the functionality up and going. Working on this all myself. I'd appreciate any info you can give on this. ![image](https://github.com/SillyTavern/SillyTavern-Extras/assets/1274076/d24b2f6d-908f-48a2-9a41-26480778ab1d)
Author
Owner

@Cohee1207 commented on GitHub (Apr 2, 2024):

The tensorboard message is purely information and appears in non-docker environments too, just ignore it

<!-- gh-comment-id:2031347358 --> @Cohee1207 commented on GitHub (Apr 2, 2024): The tensorboard message is purely information and appears in non-docker environments too, just ignore it
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
SillyTavern/SillyTavern-Extras#140
No description provided.