Kosinkadink / ComfyUI-VideoHelperSuite

Nodes related to video workflows
GNU General Public License v3.0
414 stars 74 forks source link

Error occurred when executing VHS_LoadVideo: #230

Closed petter529 closed 3 weeks ago

petter529 commented 3 weeks ago

3Lftr1wc0u image

Why do the above problems occur when loading video nodes?

Error occurred when executing VHS_LoadVideo:

local variable 'percentswap' referenced before assignment

File "D:\AI_draw\ComfyUI-aki-v1.3\execution.py", line 151, in recursive_execute output_data, output_ui = get_output_data(obj, input_data_all) File "D:\AI_draw\ComfyUI-aki-v1.3\execution.py", line 81, in get_output_data return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True) File "D:\AI_draw\ComfyUI-aki-v1.3\execution.py", line 74, in map_node_over_list results.append(getattr(obj, func)(slice_dict(input_data_all, i))) File "D:\AI_draw\ComfyUI-aki-v1.3\custom_nodes\ComfyUI-VideoHelperSuite\videohelpersuite\load_video_nodes.py", line 230, in load_video return load_video_cv(kwargs) File "D:\AI_draw\ComfyUI-aki-v1.3\custom_nodes\ComfyUI-VideoHelperSuite\videohelpersuite\load_video_nodes.py", line 144, in load_video_cv memory_limit = (psutil.virtual_memory().available + psutil.swap_memory().free) - 2 ** 27 File "D:\AI_draw\ComfyUI-aki-v1.3\python\lib\site-packages\psutil__init__.py", line 1992, in swap_memory return _psplatform.swap_memory() File "D:\AI_draw\ComfyUI-aki-v1.3\python\lib\site-packages\psutil_pswindows.py", line 262, in swap_memory percent = round(percentswap, 1)

AustinMroz commented 3 weeks ago

About a week ago, code was added to measure the memory used while loading videos and to cancel execution if this limit is reached instead of crashing ComfyUI. About 10 hours ago, I got a report that the method for checking available memory will fail on some versions of Windows. I applied a quick fix (see end of #224) in an attempt to disable this memory limit if the calculation failed, but tested it insufficiently.

I've pushed another fix, and the node should once again work for you.

petter529 commented 3 weeks ago

About a week ago, code was added to measure the memory used while loading videos and to cancel execution if this limit is reached instead of crashing ComfyUI. About 10 hours ago, I got a report that the method for checking available memory will fail on some versions of Windows. I applied a quick fix (see end of #224) in an attempt to disable this memory limit if the calculation failed, but tested it insufficiently.

I've pushed another fix, and the node should once again work for you.

Thanks,that's work OK...