Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error occurred when executing VHS_LoadVideo: #230

Closed
petter529 opened this issue Jun 13, 2024 · 2 comments
Closed

Error occurred when executing VHS_LoadVideo: #230

petter529 opened this issue Jun 13, 2024 · 2 comments

Comments

@petter529
Copy link

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
Copy link
Collaborator

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
Copy link
Author

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...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants