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

use full range by default for yuv420p if supported #6655

Merged
merged 1 commit into from
Dec 11, 2023

Conversation

21pages
Copy link
Collaborator

@21pages 21pages commented Dec 11, 2023

#6518

  1. Explicitly specify the color space as bt601
  2. Use fullrange by default for yuv420p if supported
  3. Use the pixfmt-space-range format to identify codec capabilities, make i444 proto field deprecated, and cause the non-release version of 444 true color to fail.

After the change, all colors in the image below are accurate for VP9 420 and AV1 420, but still inaccurate when r,g,b are not equal.
The conversion between yuv and rgb is irreversible, so there will be a loss of accuracy.
Teamviewer and parsec are inaccurate in both cases, anydesk accurate in both cases.
tested all codecs, 1.2.4->1.2.3, 1.2.3->1.2.4 on windows.
image

1. Explicitly specify the color space as bt601
2. Use fullrange by default for yuv420p if supported
3. Use the pix_fmt space range format to identify codec capabilities, make i444 proto field deprecated, and cause the non-release version of 444 true color to fail.

Signed-off-by: 21pages <pages21@163.com>
@21pages 21pages changed the title use fullrange by default for yuv420p if supported use full range by default for yuv420p if supported Dec 11, 2023
@rustdesk rustdesk merged commit 80afa98 into rustdesk:master Dec 11, 2023
17 of 18 checks passed
rustdesk added a commit that referenced this pull request Dec 11, 2023
rustdesk added a commit that referenced this pull request Dec 11, 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.

None yet

2 participants