Fix a bunch of path stuff

paulo-coronado / comfy_clip_blip_node

29 stars 6 forks source link

Fix a bunch of path stuff #3

Open seanlynch opened 1 year ago

seanlynch commented 1 year ago

removed use of os.chdir since that changes the directory
Get model path using folder_paths ComfyUI module
Switch all local imports to use relative syntax
Remove all manipulation of sys.path

Tested on Linux by me and on Windows by @cakeblaster on ComfyUI Matrix.

WASasquatch commented 1 year ago

Did you test BLIP though? BLIP uses relative local imports. Even using importlib and making it a dynamic package didn't work.

DoctorDiffusion commented 1 year ago

This BLIP issue broke my batch captioning workflow.

WASasquatch commented 1 year ago

This BLIP issue broke my batch captioning workflow.

I am probably going to rip BLIP out of it's package and include it in WAS-NS. Hopefully it just those couple functions it needs for what we are doing with caption and interrogation