Dynamic GOV / FPS and compression - #741
Open
cemond-genetec wants to merge 16 commits into
Open
cemond-genetec wants to merge 16 commits into
cemond-genetec wants to merge 16 commits into
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
Co-authored-by: Sriram Bhetanabottla <sriram.bhetanabottla@axis.com>
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
Contributor
|
resolve the conversations if addressed/fixed. |
bjornvolcker
reviewed
Jun 2, 2026
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
Member
|
__WG meeting: agree with proposal, but replace number of i-Frames by GOP length. |
Rephrasing as discussed in the meeting of the video working group.
bjornvolcker
approved these changes
Jun 5, 2026
sujithhanwha
approved these changes
Jul 30, 2026
| </xs:annotation> | ||
| </xs:element> | ||
| <xs:element name="MinFrameRate" type="xs:int" minOccurs="0"> | ||
| <xs:annotation> |
Contributor
There was a problem hiding this comment.
Suggest to make MinFrameRate as float, since we use float for frameratelimit.
| <para>If the whole RateControl parameter structure is missing the current state of rate control is undefined and vendor specific. A device that supports disabling of the rate control mechanisms shall reflect that by omitting the RateControl element when disabled; otherwise it shall return the current values used for RateControl. If RateControl is missing, the respective options define whether a RateControl element can be (re-)added.</para> | ||
| <para>The resolution's width and height are based on the Bounds field of the VideoSourceConfiguration. Therefore, if there is a 90-degree rotation, the width will represent the height of the encoded video, and the height will represent the width of the encoded video.</para> | ||
| <para>To further reduce bitrate, the device may lower encoding quality in regions of lesser importance, controlled by the EncodingStrategy parameter. Values are defined in EncodingStrategiesSupported. Encoding efficiency can be enhanced by enabling DynamicGov and DynamicFps, while it can be constrained using MaxGovLength and MinFramerate.</para> | ||
| <para>The resolution's width and height are based on the Bounds field of the VideoSourceConfiguration. Therefore, if there is a 90-degree rotation, the width will represent the height of the encoded video, and the height will represent the width of the encoded video.</para> |
Contributor
There was a problem hiding this comment.
Better to mention this in the description "MaxGovLength should strictly be > GovLength, and MinFrameRate should strictly be > 0"
There was a problem hiding this comment.
Reworded this a bit to include these details
Added more details to the description Fixed XML merge issue
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
8 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add missing configuration for advanced bitrate reduction technologies
Last add EncodingEffortLevel - control how the device can reduce quality on lower interest region in order to reduce bandwidth. Dynamic GOV and FPS have more impact when effort level is high.