lllyasviel / stable-diffusion-webui-forge

GNU Affero General Public License v3.0
8.33k stars 811 forks source link

Operations handling VAE / Text Encoders #1532

Open altoiddealer opened 2 months ago

altoiddealer commented 2 months ago

Edit- Most of these sub issues have been solved. Outstanding issues are at the top


I'm making this as a broad topic for the functions related to "VAE" and "Text Encoders".

According to Illyasviel, these are now one and the same in Forge.

If anyone makes comments for related issues, I'll add to the OP.

I've attempted to do these things myself, but they were over my head (except the last item).


Sending PNGInfo to txt2img / img2img

Module 1, Module 2, Module 3, etc - these settings should automatically populate the VAE/Text Encoders multi-select menu.

Or, in Extra Parameters field that can appear below the main sliders

Screenshot 2024-08-25 204623

XYZ Parameters

Currently, only "VAE" can be used via XYZ.

Screenshot 2024-08-25 205610

✅ Checkpoint Preferences

Original Issue Currently no way to set default Text Encoders for checkpoints. Screenshot 2024-09-08 102511

✅ API override_settings

Original Issue Ideally, a `list` of modules (VAEs and Text Encoders) should be able to be provided in the `override_settings` dict. ![361053476-bb400d93-2080-4a03-8b0e-d3d815ef6337](https://github.com/user-attachments/assets/c90ddd89-d5a2-40f9-99bd-50eec0b073af)

✅ Get all VAE / Text Encoders via API

Already done and merged - https://github.com/lllyasviel/stable-diffusion-webui-forge/pull/1463

Screenshot ![361048886-875d9158-72bb-4b51-8668-355309df56ac](https://github.com/user-attachments/assets/d08ab392-2f7f-4ecf-92c9-d9a69682a72a)
altoiddealer commented 1 month ago

Added Checkpoint Preferences to the list of things that need to be updated.

@Panchovix Sorry to ping you, but I feel some of these should be a high priority fix. I've attempted a few times to make any sort of progress with resolving these, but I don't know enough backend stuff to even figure out where to begin.

altoiddealer commented 1 month ago

I can confirm that Flux is working via the API.

I noticed that there is now an override_settings key where you can specify the forge_additional_modules. I updated the OP