expose the go2rtc instance to avoid multiple go2rtc instances and high loads #564
regevbr
started this conversation in
Feature Requests
Replies: 1 comment 1 reply
-
When I first switched to go2rtc I had originally planned to expose it, and, in the first few beta releases of 5.1.0, it was possible to use the go2rtc instance in the ring-mqtt addon directly, without the additional go2rtc addon, but after testing with various scenarios, I decided it just wasn't worth it for a multitude of reasons:
Basically, I see little reason to take on the headache of maintaining all of that in ring-mqtt when I only need the rtsp-proxy portion of go2rtc, all for the benefit of maybe 20MB of RAM and <5% of a single core of CPU per stream. |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
While this library uses go2rtc to convert WebRTC to RTSP, it also recommends (in the FAQ) using go2rtc to convert the RTSP stream back to WebRTC using go2rtc for better performance. This has 2 outcomes:
Since your go2rtc configuration requires running local scripts, I can't see a way to utilize an external addon. So what I suggest is a way to expose the go2rtc instance completely (including UI) using the host network (might need to release a different addon version for that). This way we can use the go2rtc of this addon for other streams, and use directly the webrtc of ring, instead of double converting it (using the amazing WebRTC component, without creating a generic camera integration (that indeed must use an RTSP stream).
To achieve this change, there is also a need to mount the go2rtc config from the local disk (so users can edit it) and make the code read the config and edit it inline (instead of overwriting it).
Thoughts?
Beta Was this translation helpful? Give feedback.
All reactions