-
Notifications
You must be signed in to change notification settings - Fork 667
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
❌ [Experimental, not stable] Add websocket #2370
base: main
Are you sure you want to change the base?
Conversation
Why no merge ? It seems clean code after a small review. |
When I tested it last, it did not work stable, and I was unable to find out why. Maybe in winter I will have more time again to look into it but not now. |
90% sur it's related to your psram implementation. free_psram_heap() = unknown, no include and it's not an usual esp-idf function. The psram need to be allocated with heap_caps_malloc(). |
Well, you didn't really look into the code, right? A quick search would have shown you that it is a wrapper I wrote for debugging: I strongly suggest you first study https://github.com/jomjol/AI-on-the-edge-device/blob/rolling/code/components/jomjol_helper/psram.cpp before giving any claims! |
This PR adds websocket support. With this we can make the web UI more responsive.
Call
http://<IP>/websocket.html
for a test:Notes
Nevertehless some extra internal memory is needed due to the increased
config.max_uri_handlers
.