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

Fix gray tint. #1

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Fix gray tint. #1

wants to merge 4 commits into from

Conversation

rkfg
Copy link

@rkfg rkfg commented Sep 13, 2023

I copied the code from the upstream and fixed it too. In the upstream the DDIM parameters are only set if the DDIM sampler is used for generation but until then the gray noise is always present. However, as soon as you use DDIM once it's fixed for all samplers (until you restart SD that is). So I simply changed the code to always execute it.

JaredTherriault and others added 4 commits August 28, 2023 10:24
-Set mm_sd_v14 as the default instead of v15, as most end users tend to prefer it
-Load actual motion modules from the model folder instead of hardcoding v14 and v15 in the dropdown
-Add support for webp and apng file formats
-Add quality sliders and lossless options for webp and apng
-Add ability to format the file names
-Add ability to output to a custom folder, along with named subdirectories similar to the feature set for images
-Brought injection code up to parity with Kosinkadink's ComfyUI's implementation
-Updated motion_module
-Updated groupnorm hack
-Removed apng option, it just doesn't work well in imageio's implementation, may return in future
-Add option to always save videos
-Add infotext to video generations
-Set UI parameters from infotext when pasting/restoring from params.txt
-Separated settings into separate py file
-Updated info for options
-made some function names more descriptive and moved some code around for readbility
-Make it so the infotext shown underneath the video after a generation reflects the actual video, not just main prompt
-Fixes an issue that would occur when trying to generate images after generating video. No longer asserts.
@JaredTherriault
Copy link
Owner

JaredTherriault commented Sep 15, 2023

Hey, thanks for the heads up! Would you be able to close this PR, pull latest, then reimplement the changes in a new commit? This change actually reverts a lot of recent changes due to the aggregation of commits. I could also just remove the DDIM check myself, but I want you to have the credit for the find and fix.

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.

3 participants