comfyanonymous / ComfyUI

The most powerful and modular diffusion model GUI, api and backend with a graph/nodes interface.
https://www.comfy.org/
GNU General Public License v3.0
58.13k stars 6.17k forks source link

Comfyui shows high-quality previews? #3551

Open A-B-01 opened 6 months ago

A-B-01 commented 6 months ago

Hi, after the latest updates, I noticed that the "K sampler preview" seems to generate corrupted images

corrupted preview

comfyanonymous commented 6 months ago

I changed the "auto" previews to the low quality ones, if you want the high quality ones you need to use "taesd" instead of "auto".

A-B-01 commented 6 months ago

I changed the "auto" previews to the low quality ones, if you want the high quality ones you need to use "taesd" instead of "auto".

--preview-method taesd

Great! It worked!

Thanks a lot!!!

ltdrdata commented 6 months ago

I changed the "auto" previews to the low quality ones, if you want the high quality ones you need to use "taesd" instead of "auto".

--preview-method taesd

Great! It worked!

Thanks a lot!!!

FYI, the default setting changing to low from taesd is because taesd causes significant performance loss.

shawnington commented 6 months ago

I changed the "auto" previews to the low quality ones, if you want the high quality ones you need to use "taesd" instead of "auto".

--preview-method taesd Great! It worked! Thanks a lot!!!

FYI, the default setting changing to low from taesd is because taesd causes significant performance loss.

Thats very confusing naming with the command line options, going from normal being high quality to now you have to know what taesd is to re-enable it.

but also, its very questionable to have gui options be command line, when it should just be a toggle in the gui. No reason there can't be a toggle for quality.

Some of the launch options are quite cryptic, which if you have no problems looking around the code base isn't an issue, but for your average person, some of them, are a bit... googling will make you even more confusedish.

ltdrdata commented 6 months ago

I changed the "auto" previews to the low quality ones, if you want the high quality ones you need to use "taesd" instead of "auto".

--preview-method taesd Great! It worked! Thanks a lot!!!

FYI, the default setting changing to low from taesd is because taesd causes significant performance loss.

Thats very confusing naming with the command line options, going from normal being high quality to now you have to know what taesd is to re-enable it.

but also, its very questionable to have gui options be command line, when it should just be a toggle in the gui. No reason there can't be a toggle for quality.

Some of the launch options are quite cryptic, which if you have no problems looking around the code base isn't an issue, but for your average person, some of them, are a bit... googling will make you even more confusedish.

So the ComfyUI-Manager supports opitons for preview.

wisniew commented 5 months ago

This should be addressed at least in the readme.

andreszs commented 1 month ago

After the last update v0.2.3-13-g7390ff3, the --preview-method taesd option persists in rendering blurry, noisy previews instead of hight quality ones. This is after restarting comfyui and the browser countless times.

github-actions[bot] commented 1 week ago

This issue is being marked stale because it has not had any activity for 30 days. Reply below within 7 days if your issue still isn't solved, and it will be left open. Otherwise, the issue will be closed automatically.