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

Dynamic preview method per task #6205

Open
huchenlei opened this issue Dec 24, 2024 · 2 comments
Open

Dynamic preview method per task #6205

huchenlei opened this issue Dec 24, 2024 · 2 comments
Labels
Feature A new feature to add to ComfyUI.

Comments

@huchenlei
Copy link
Collaborator

Feature Idea

Each queued task can spcify its own preview method instead of relying on a static preview method set via server launch args.

This should enables selecting preview method by adjusting frontend settings without restart the server.

Existing Solutions

https://github.com/ltdrdata/ComfyUI-Manager has a UI selection to pick preview method. However, it requires restart to take effect.

image

Other

No response

@huchenlei huchenlei added the Feature A new feature to add to ComfyUI. label Dec 24, 2024
@ltdrdata
Copy link
Collaborator

ltdrdata commented Dec 25, 2024

The preview method selection in ComfyUI-Manager is applied immediately without restarting the server.
However, it does not affect the inference currently in progress and will only apply to the next inference.

@huchenlei
Copy link
Collaborator Author

The preview method selection in ComfyUI-Manager is applied immediately without restarting the server. However, it does not affect the inference currently in progress and will only apply to the next inference.

I think this feature should be in core. Maybe you can help upstream the manager impl to core?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature A new feature to add to ComfyUI.
Projects
None yet
Development

No branches or pull requests

2 participants