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

limit video helper meta data to the modes we will need #2226

Closed
totaam opened this issue Mar 20, 2019 · 3 comments
Closed

limit video helper meta data to the modes we will need #2226

totaam opened this issue Mar 20, 2019 · 3 comments
Labels

Comments

@totaam
Copy link
Collaborator

totaam commented Mar 20, 2019

Issue migrated from trac ticket # 2226

component: encodings | priority: major | resolution: wontfix

2019-03-20 04:21:14: antoine created the issue


Typically, on Linux with X11 we will only need BGRX so there is no need to keep track of the conversions to and from BGR, RGB or RGBX, etc

@totaam
Copy link
Collaborator Author

totaam commented Mar 21, 2019

2019-03-21 12:01:16: antoine changed status from new to closed

@totaam
Copy link
Collaborator Author

totaam commented Mar 21, 2019

2019-03-21 12:01:16: antoine set resolution to wontfix

@totaam
Copy link
Collaborator Author

totaam commented Mar 21, 2019

2019-03-21 12:01:16: antoine commented


Too difficult, most combinations are valid, they're just not very useful.

ie: when we have BGRX to h264 using x264 (or nvenc) then that's what we'll use, but going via YUV420P and a CSC step is also a valid option (which uses less bandwidth and CPU). We never want to use XRGB to h264, because that would require a BGRX to XGRB CSC step, but we don't know in advance that there are going to be better options..

@totaam totaam closed this as completed Mar 21, 2019
@totaam totaam added the v2.5.x label Jan 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant