[Errno 32] Broken pipe - To much meta data ? about comfyui-videohelpersuite HOT 9 CLOSED

I've found the issue and I'll push the fix in the next couple minutes once I've done my sanity checks.

As you guessed initially, when it saves with longer metadata, it uses a different code path. While the logic related to metadata isn't at issue here, the crf information gets added a second time in the command where it shouldn't.

from comfyui-videohelpersuite.

from comfyui-videohelpersuite.

Not currently, but there was an earlier issue asking about it as well. I'll look into adding it in the future.

from comfyui-videohelpersuite.

Metadata wasn't among the changes that was made today, so I have my doubts that it would be related. I'll probably look into lowering the severity of the metadata warning since I've not seen any actual issues caused by it.

The error message provided tells me that the ffmpeg process closed with some error, but doesn't go far enough back to give me any insight on what the issue is. Can you provide earlier logs?

from comfyui-videohelpersuite.

full stack

from comfyui-videohelpersuite.

Thanks, I'm looking into it. Can you check if it breaks with other video formats selected?

from comfyui-videohelpersuite.

Its something with my workflow I created because other workflows work.. i have no idea.. i ripped everything out.. I can post the workflow also.. I could be an idiot and doing something wrong. lol :)

from comfyui-videohelpersuite.

Is their a way to opt out of meta data if you dont want to add it, im assuming your adding the workflow..

from comfyui-videohelpersuite.

A recent PR makes the Broken pipe due to metadata no longer causs everything to break down, and simply retries without the metadata. A save_metadata toggle is also present. Closing the issue, since should have been fixed for a several days now 👍

from comfyui-videohelpersuite.