Skip to content
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

sync #1

Open
wants to merge 1,343 commits into
base: master
Choose a base branch
from
Open

sync #1

wants to merge 1,343 commits into from

Conversation

neonsecret
Copy link
Owner

No description provided.

ZeroCool940711 and others added 30 commits November 23, 2022 19:01
Initial support for the Stable Diffusion 2.0 model. 
As of right now changing the `default_model_config` option on the
`webui_streamli.yaml` to point to the `v2-inference.yaml` file and
downloading manually the Stable Diffusion v2.0 model and placing it on
the `models/custom` folder will let you load it and run it but the
colors are still messed up, needs more work on it to fix this issue.
…#1687)

# Description
made factory functions to parse config and generate settings page.
# Checklist:

- [x] I have changed the base branch to `dev`
- [x] I have performed a self-review of my own code
- [x] I have commented my code in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
was using them with pprint for troubleshooting, but no longer need.  better checks are/will be in place when more gets hammered out.
…ing users to render bigger resolutions without running out of memory.
…to be set even if empty.

Added shutup as a dependency to shutup python warnings for good.
…ing users to render bigger resolutions without running out of memory. (#1693)

- Fixed issue with new ldm folder requiring the personalization_config
to be set even if empty.
- Added shutup as a dependency to shutup python warnings for good.
# Description
Frame generation is being done in chunks of the number of samples
chosen, so when you needed 120 frames but set 200 samples, it would
generate 200 frames.

* Added a break to stop frames from generating once total_frames is
reached

# Checklist:

- [x] I have changed the base branch to `dev`
- [x] I have performed a self-review of my own code
- [ ] I have commented my code in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
ZeroCool940711 and others added 30 commits July 26, 2023 17:11
…previous one does not exist anymore or is outdated.
…previous one does not exist anymore or is outdated. (#1810)
…s pre-commit removed their imports for some reason after I fixed it before.
…s pre-commit removed their imports for some reason after I fixed it before. (#1814)
# Description

Please include:
I'm not sure why this `docker-compose.yml` was deleted in the first
place. Nearly half of the instructions in the readme for this repo rely
on this file being there.

Essentially, it's just another way to start up the container using
`docker compose` . This makes the container much more usable for power
users, as they'd be able to configure volumes much easier. Also, instead
of having to remember the whole complicated `docker container run --rm
-d -p 8501:8501 -e STREAMLIT_SERVER_HEADLESS=true -e
"WEBUI_SCRIPT=webui_streamlit.py" -e "VALIDATE_MODELS=false" -v
"${PWD}/outputs:/sd/outputs" --gpus all tukirito/sygil-webui:latest`
command, it's as simple as running `docker compose start` or `up`, since
all of the configs are in the compose file, you wouldn't need anything
else. The compose file has been tested and runs without any issues. The
readme instructions can stay mostly as is.

Additionally, as mentioned before, the readme instructions rely heavily
on docker compose. If the repo creators wish to not include docker
compose, then it should all be omitted from the readme, as it makes this
extremely confusing for any user.

Closes: # (issue)

# Checklist:

- [ ] I have changed the base branch to `dev`
- [ ] I have performed a self-review of my own code
- [ ] I have commented my code in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
# Description

Please include:
* relevant motivation
* a summary of the change
* which issue is fixed.
* any additional dependencies that are required for this change.

Closes: # (issue)

# Checklist:

- [x] I have changed the base branch to `dev`
- [x] I have performed a self-review of my own code
- [ ] I have commented my code in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
# Description

Please include:
* relevant motivation
* a summary of the change
* which issue is fixed.
* any additional dependencies that are required for this change.

Closes: # (issue)

# Checklist:

- [ ] I have changed the base branch to `dev`
- [ ] I have performed a self-review of my own code
- [ ] I have commented my code in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
# Description
Give entrypoint.sh executable permission to fix docker error.

Closes: #1479

# Checklist:

- [*] I have changed the base branch to `dev`
- [*] I have performed a self-review of my own code
- [ ] I have commented my code in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
# Description

For context, modelId is getting deprecated
huggingface/huggingface_hub#2408
Closes: # (issue)

# Checklist:

- [x] I have changed the base branch to `dev`
- [x] I have performed a self-review of my own code
- [x] I have commented my code in hard-to-understand areas
- [x] I have made corresponding changes to the documentation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.