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

The video size of camera is not correct to 1920x1080 #2845

Closed
12 tasks
iesv opened this issue Dec 26, 2023 · 2 comments
Closed
12 tasks

The video size of camera is not correct to 1920x1080 #2845

iesv opened this issue Dec 26, 2023 · 2 comments

Comments

@iesv
Copy link

iesv commented Dec 26, 2023

Which version are you using?

v1.4.1

Which operating system are you using?

  • Linux amd64 standard
  • Linux amd64 Docker
  • Linux arm64 standard
  • Linux arm64 Docker
  • Linux arm7 standard
  • Linux arm7 Docker
  • Linux arm6 standard
  • Linux arm6 Docker
  • [x ] Windows amd64 standard
  • Windows amd64 Docker (WSL backend)
  • macOS amd64 standard
  • macOS amd64 Docker
  • Other (please describe)

Describe the issue

The video size of camera is not correct to 1920x1080 when publishing with chrome of Android phone.

Describe how to replicate the issue

  1. start the server
  2. publish with chrome of Android: https://192.168.10.10:8889/live/publish
  3. set my phone to shoot in landscape mode
  4. read with chrome of PC: http://127.0.0.1:8888/live
  5. the video size of camera is not correct to 1920x1080

Did you attach the server logs?

no

Did you attach a network dump?

no

aler9 added a commit that referenced this issue Apr 16, 2024
@aler9
Copy link
Member

aler9 commented Apr 16, 2024

Hello, after some research it was discovered that the framerate, width and height specified in the WebRTC publish page are not enforced, but used as reference. The actual width, height and framerate are picked from a set of fixed values, that depends on the specific camera, and are chosen to be as similar as possible to the reference ones. This is a WebRTC behavior that cannot be changed.

In order to avoid confusion i've renamed "video width" into "video width (ideal)" and so on (#3254).

@aler9 aler9 closed this as not planned Won't fix, can't repro, duplicate, stale Apr 16, 2024
aler9 added a commit that referenced this issue Apr 17, 2024
Copy link
Contributor

This issue is mentioned in release v1.8.0 🚀
Check out the entire changelog by clicking here

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

No branches or pull requests

2 participants