search

Kosinkadink / ComfyUI-VideoHelperSuite

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

server.py missing .utils and time imports, a well as inputs into get_sorted_dir_files_from_directory #94

Closed Kosinkadink closed 8 months ago

Kosinkadink commented 8 months ago

I think some stuff might have gotten mixed up in the server.py file - the import issues are a quick fix, but I'm not sure where directory, skip_first_frames, and select_every_nth would be obtained from the API call, so I'll leave it to you.

image image

Kosinkadink commented 8 months ago

Oh yeah, I should have mentioned: this causes an error to be logged in the console (I think the node still work though) when Load Images (Path) is used and the path is filled in.


Traceback (most recent call last):
  File "f:\ComfyUI\venv\lib\site-packages\aiohttp\web_protocol.py", line 433, in _handle_request
    resp = await request_handler(request)
  File "f:\ComfyUI\venv\lib\site-packages\aiohttp\web_app.py", line 504, in _handle
    resp = await handler(request)
  File "f:\ComfyUI\venv\lib\site-packages\aiohttp\web_middlewares.py", line 117, in impl
    return await handler(request)
  File "F:\ComfyUI\server.py", line 46, in cache_control
    response: web.Response = await handler(request)
  File "F:\ComfyUI\custom_nodes\ComfyUI-VideoHelperSuite\videohelpersuite\server.py", line 66, in view_video
    valid_images = get_sorted_dir_files_from_directory(directory, skip_first_images, select_every_nth, FolderOfImages.IMG_EXTENSIONS)
NameError: name 'get_sorted_dir_files_from_directory' is not defined```