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

URGENT NEED: Improve camera feed #8040

Closed
1 of 3 tasks
aparnacoronasafe opened this issue Jun 18, 2024 · 1 comment · Fixed by #8140
Closed
1 of 3 tasks

URGENT NEED: Improve camera feed #8040

aparnacoronasafe opened this issue Jun 18, 2024 · 1 comment · Fixed by #8140
Assignees

Comments

@aparnacoronasafe
Copy link
Member

aparnacoronasafe commented Jun 18, 2024

Is your feature request related to a problem? Please describe.
The new camera feed component is getting a lot of negative feedbacks for the following reason:

  • The aspect ratio has changed. The width is increased with respect to the height. This reduces the feed space on phone screens.
  • Is it possible that the new component is slower and requires a better internet speed to work? Many complaints of camera feed freezing with internet speed dips. Users say this was not the case before
  • Enhancements to fullscreen view in mobiles for camera feed #8085
@rithviknishad
Copy link
Member

@aparnacoronasafe

The aspect ratio has changed.

The aspect ratio has not been changed. It uses 16:9 standard video ratio before and now. Could you provide screenshots / more details for it?

Is it possible that the new component is slower and requires a better internet speed to work.

The new component does not require any extra internet speed to work. It uses the same method as before to stream the video.

The new component would attempt to reconnect if the latency goes beyond 5 seconds and would cause a momentary black screen until reconnected. Should we consider increasing the latency threshold?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants