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

Replace Http Server + Add WebSockets #9

Merged
merged 9 commits into from
Feb 13, 2023

Conversation

pythongosssss
Copy link
Collaborator

As discussed, implemented WebSockets to replace the current polling.
I was going to move the classes out to separate files, but for ease of reviewing i've left them in main but it might be worth you splitting them out

@comfyanonymous
Copy link
Owner

Seems to work and the good news is that I noticed that aiohttp seems to already get pulled in by one of the dependencies.

Small issue that I found is that I need to do CTRL-C twice to kill it now?

webshit/index.html Outdated Show resolved Hide resolved
main.py Outdated Show resolved Hide resolved
@pythongosssss
Copy link
Collaborator Author

Investigating the CTRL+C issue, I don't have it on Windows, trying to get it running via WSL

main.py Outdated Show resolved Hide resolved
This was for hiding an error on Windows, but it isn't required
@comfyanonymous comfyanonymous merged commit 68a1ac7 into comfyanonymous:master Feb 13, 2023
Omer-ler pushed a commit to uglabs/ComfyUI that referenced this pull request Nov 14, 2023
Added a function to load the directories in the optional file: extra_model_paths.yaml
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

Successfully merging this pull request may close these issues.

2 participants