-
Notifications
You must be signed in to change notification settings - Fork 169
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
Add support for H264 profiles #1542
Conversation
PS: For some reason the segmenter test case is failing - I guess it's unrelated to my changes because it fails on my local |
{"name": "prof2", "bitrate": 765, "fps": 876, "fpsDen": 12, "width": 456, "height": 987}, | ||
{"name": "passthru_fps", "bitrate": 890, "width": 789, "height": 654}]}`) | ||
{"name": "passthru_fps", "bitrate": 890, "width": 789, "height": 654, "profile": "H264ConstrainedHigh"}]}`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TeSt FoR mIxEd CaSe?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually it already is mixed case....!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚢
Added some back-compat regression tests here https://gist.github.com/j0sh/9d884a19bb92e504092e71fd19d812e5#file-profiles-standalone-sh and https://gist.github.com/j0sh/9d884a19bb92e504092e71fd19d812e5#file-profiles-sh , things seem to work nicely.
What does this pull request do? Explain your changes. (required)
This PR adds support for specifying the H264 encoder profile (#1531) by passing it as a field in the VideoProfile related structs.
Specific updates (required)
net.VideoProfile
struct - Serialize/Deserialize to LPMS VideoProfileFullProfiles3
field only when the new Profiles field is usedHow did you test each of these updates (required)
Does this pull request close any open issues?
Fixes #1531
Checklist:
./test.sh
pass