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

Camera connection - use substream instead of mainstream #11

Merged
merged 1 commit into from
May 14, 2023

Conversation

calebolson123
Copy link
Owner

This PR switches the sample connection string to use substream, instead of the mainstream. I recently discovered that this exists, and upon testing, increases stability of the live feed processing service significantly (haven't seen it go down yet).

There are some known issues regarding live-mode's stability, where after a period of time, the service will crash with errors similar to what's described here.

Using substream instead of mainstream reduces the bandwidth of the stream, increasing stability, while maintaining clear enough images for the core computer vision logic to recognize baby features over time.

An upcoming PR largely overhauling the design replaces this line, and a note about the difference between mainstream and substream ideally would be preserved in (or after) that PR so that users know about the now preferred substream channel.

…ry, this increases stability while processing in live-mode
@calebolson123 calebolson123 merged commit 8bf1916 into master May 14, 2023
@calebolson123 calebolson123 deleted the camera-use-substream-connection branch May 14, 2023 13:46
@calebolson123 calebolson123 mentioned this pull request May 14, 2023
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.

1 participant