mirror of
https://github.com/SillyTavern/SillyTavern-Extras.git
synced 2026-10-16 18:41:52 +00:00
[GH-ISSUE #231] Requirement file rework not reflected for docker #140
Labels
No labels
bug
duplicate
planned
pull-request
under consideration
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
SillyTavern/SillyTavern-Extras#140
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
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
@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.
@MNarath1 commented on GitHub (Mar 30, 2024):
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.
@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
@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.
@Cohee1207 commented on GitHub (Apr 2, 2024):
The tensorboard message is purely information and appears in non-docker environments too, just ignore it