vinavfx / ComfyUI-for-Nuke

API to be able to use ComfyUI nodes within nuke, only using the ComfyUI server
GNU General Public License v3.0
193 stars 17 forks source link

'utf-32-be' codec can't decode bytes #4

Closed screameight closed 1 month ago

screameight commented 7 months ago

During the submit process, I have the following issue about the websocket :

"ERROR:websocket:error from callback <function progress..on_message at 0x000002B5677BEF80>: 'utf-32-be' codec can't decode bytes in position 8-11: code point not in range(0x110000)"

What does that mean ?

vinavfx commented 7 months ago

Does ComfyUi work for you on the web?

screameight commented 7 months ago

Yes perfectly

vinavfx commented 7 months ago

Can you give me that nuke info:

import websocket print(sys.version) print(nuke.NUKE_VERSION_STRING) print(websocket.version)

is it windows? What comfyui commit are you using?

I need that information to be able to duplicate the error since it works well for me. thank you

screameight commented 7 months ago

Hello Francisco,

Here is the needed informations :

Sys version : 3.9.10 [MSC v.1929 64 bit (AMD64)] Nuke Version : 14.1.4 Websocket version : 1.8.0

Yes it's Windows 11 with this ComfyUI repo : https://github.com/comfyanonymous/ComfyUI How can I see the comfyUI commit ?

Thank you :)

vinavfx commented 7 months ago

cd ComfyUI git log

The first one at the top is the commit you should be using

screameight commented 7 months ago

I think it’s that : commit 6c6a39251fe313c56a88c90d820009073b623dfe (HEAd -> master, origin/master) Author: comfyanonymous Date: Tue Apr 2 11:46:34 2024 -0400

thank you !

vinavfx commented 7 months ago

Thanks, when I try it I'll tell you

Zemram commented 2 months ago

Try using a 'VAEDecode' for XL models, not sdVaeEncode.

vinavfx commented 2 months ago

https://github.com/vinavfx/ComfyUI-for-Nuke/pull/24

vinavfx commented 1 month ago

@claussteinmassl already fixed this problem, but investigating I saw that in older versions of Nuke only bytes arrive in the websocket message so returning it by data type does not work, the only solution I found for greater compatibility is to just use try, except so that it does not throw this error, I ask you to support me by testing it to close this unless you have another solution. @screameight

https://github.com/vinavfx/ComfyUI-for-Nuke/commit/b50ee16809a6cfaf11889476759c4f2af2c4eeac