ComfyUI icon indicating copy to clipboard operation
ComfyUI copied to clipboard
Published 5 days ago • verified publisher icon comfyanonymous

PNG cICP chunk saving support

Open catboxanon opened this issue 11 months ago • 2 comments

Addresses https://github.com/comfyanonymous/ComfyUI/issues/6505

catboxanon avatar Jan 30 '25 20:01 catboxanon

Latest Pillow release (11.2.1) now supports the cICP chunk, so I've removed the implementation hacks related to it.

catboxanon avatar Apr 14 '25 16:04 catboxanon

What is needed to be done in order to get this merged?

I have tested this and can confirm it works as intended. PNG files gets saved with the cICP data from the VAE. In the UI it also shows up with the correct color, but without any HDR effect. AFAIK, there is no support in browsers for HDR in HTML canvas, but it showing up with the correct colors and SDR rendering indicates it does receive the cICP data. So I believe this is the best we can do with current browser tech.

spillerrec avatar Jun 07 '25 10:06 spillerrec