Added shared.cmd_opts.data_dir prefix to "outputs" and "log" paths.
Description
Added shared.cmd_opts.data_dir prefix to "outputs" and "log" paths.
Fixes the issue where hardcoded default "outputs" and "log" paths may not be writable and enables placing them under --data-dir option path.
Screenshots/videos:
NA
Checklist:
- [X] I have read contributing wiki page
- [X] I have performed a self-review of my own code
- [X] My code follows the style guidelines
- [X] My code passes tests
related conversation https://github.com/AUTOMATIC1111/stable-diffusion-webui/pull/14442#issuecomment-1870607628
lol didn't realize you basically have the same idea as me
I made an alternative implementation with some changes see PR https://github.com/AUTOMATIC1111/stable-diffusion-webui/pull/14446
namely truncating the path to CWD when possible (so if one is not using --data-dir it behave the same as before)
I added you as co-author
I merged the other one. Just was wew proposes, I'd rather not absolute paths to config by default.