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

[Feature Request]: Add Preset Information to the Log #72

Open
1 task done
DavidDragonsage opened this issue Jan 30, 2025 · 2 comments
Open
1 task done

[Feature Request]: Add Preset Information to the Log #72

DavidDragonsage opened this issue Jan 30, 2025 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@DavidDragonsage
Copy link
Owner

DavidDragonsage commented Jan 30, 2025

Is there an existing issue for this?

  • I have searched the existing issues and checked the recent builds/commits

What would your feature do?

Currently, diffuser based AI generators such as Kolors provide almost no useful information in the log. The Base Model is simply listed as "None". The only hint that Kolors was used is that it is the Backend Engine is "Kolors+", but this does not tell the user which variation of Kolors was in effect.

Proposed workflow

  1. Add a "Preset Name" parameter to the log
  2. This may have to be filtered when the log is pasted into the prompt field as a means of image replication

Additional information

In fact there are three presets which use the installed Kolors package in different ways: Kolors (standard), KolorsFast and KolorsTile. The main difference between Kolors (standard) and KolorsFast is the use of "Hyper-SDXL-8steps-lora.safetensors".

The specific condition of using this LoRA could be detected, but I think that the more general approach of always providing Preset information in the log would be more useful.

@DavidDragonsage DavidDragonsage added the enhancement New feature or request label Jan 30, 2025
@iwr-redmond
Copy link
Collaborator

This can be resolved in the future by replacing the Diffusers wrapper node with the ComfyUI native node, which was not ready for mainstream use when Simple 0916 was released.

@DavidDragonsage
Copy link
Owner Author

Well that is good to know! That sounds like this could be combined with the scheduled Comfy upgrade.

But apart from that, I think it would be useful in general to include the Preset information in the log.

@DavidDragonsage DavidDragonsage self-assigned this Mar 1, 2025
@DavidDragonsage DavidDragonsage changed the title [Feature Request]: Provide Useful Log Information When Using Diffusers [Feature Request]: Add Preset Information to the Log Mar 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants