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

Add ays, clip skip and vae, to image embedded params #316

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

one-lithe-rune
Copy link

Motivation

Since my own little frontend app reads the embedded parameter info from PNG images to display them ready for editing and generating again, I noticed that the Sampler tag includes 'karras' if you have set that scheduler, but doesn't include 'ays' if you have set that. Since 'ays' has been added recently I'm guessing this is unintentional?

Since I was already in there to fix that, I also added VAE and Clip Skip to be written if vae_path or clip_skip have been set, since those are also useful for my app.

Changes

  • If 'ays' is set as the scheduler include it in the 'Sampler' tag in the data embedded into the final image. This makes it the same as how Karras is handled
  • If a custom VAE path is set, include the vae name (without path and extension, using the same function as for Model) in embedded image params under a VAE: tag.
  • If a custom Clip skip is set, include that Clip skip value in embedded image params under a Clip skip: tag.

- If 'ays' is set as the scheduler include it in the 'Sampler' tag in the data
embedded into the final image. This makes it the same as how Karras is handled
- If a custom VAE path is set, include the vae name (without path and extension)
in embedded image params under a `VAE:` tag.
- If a custom Clip skip is set, include that Clip skip value in embedded image
params under a `Clip skip:` tag.
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.

1 participant