Dynamic GOV / FPS and compression

[cemond-genetec]

No description provided.

[bsriramprasad]

Suggested change

	<xs:documentation>Indicates that device is allowed to skip unnecessary P-frames when there is no motion. To ensure a minimum framerate, enable GuaranteedFrameRate and set FrameRate.</xs:documentation>
	<xs:documentation>Indicates that device is allowed to skip unnecessary P-frames. To ensure a minimum framerate, enable GuaranteedFrameRate and set FrameRate.</xs:documentation>

[bsriramprasad]

As there will always be some motion and depends on what we classify as motion and it gets into the details, to avoid such ambiguity, we recommend to remove 'no motion'.

[bsriramprasad]

Additional Feedback from Axis

• DynamicEncoding needs to be part of VideoRateControl2 (better as separate/individual parameters/attributes than another sub-structure as DynamicEncoding
  • These are treated as 'controlling levers' than configuration itself.
• Need to add below configuration parameters for VideoEncoder2Configuration that are affected by above levers
  • Feature DynamicGov
    • Existing GovLength parameter need to be repurposed as Minimum GovLength
    • Need a new parameter for MaxGovLength
• Feature Dynamic FPS
  • Maximum FPS set via FrameRateLimit
  • Need a new parameter for Minimum FPS
  • Recommend NOT to tie this feature to GuaranteedFrameRate as the primary use case for that is like a fixed/constant framerate and device adapts all internal parameters including bitrate to get there.

[cemond-genetec]

Additional Feedback from Axis

• DynamicEncoding needs to be part of VideoRateControl2 (better as separate/individual parameters/attributes than another sub-structure as DynamicEncoding

  • These are treated as 'controlling levers' than configuration itself.

• Need to add below configuration parameters for VideoEncoder2Configuration that are affected by above levers

  • Feature DynamicGov

    • Existing GovLength parameter need to be repurposed as Minimum GovLength
    • Need a new parameter for MaxGovLength

• Feature Dynamic FPS

  • Maximum FPS set via FrameRateLimit
  • Need a new parameter for Minimum FPS
  • Recommend NOT to tie this feature to GuaranteedFrameRate as the primary use case for that is like a fixed/constant framerate and device adapts all internal parameters including bitrate to get there.

I will wait the others opinion next week. It is inline with the original proposition with 2 new parameters for MaxGovLength and MinimumFps because existing parameter are needed for the upper bound (for bandwidth control).

1. Dynamic encoding parameters to be part of VideoRateControl2 makes sense to me
2. I agree
3. I agree.
   Should the GuaranteedFrameRate be disabled (automatically) when DynamicFPS is used ?

[kieran242]

@cemond-genetec can you please add a description to the PR please.