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

[encode] Allow GPB set to off for HEVC encode #21

Merged
merged 1 commit into from
Jun 4, 2024
Merged

[encode] Allow GPB set to off for HEVC encode #21

merged 1 commit into from
Jun 4, 2024

Conversation

lsun30
Copy link
Contributor

@lsun30 lsun30 commented May 31, 2024

Some CTS tests of android.videocodec.cts.VideoEncoderMaxBFrameTest# testMaxBFrameSupport failed with "Number of BFrames in a SubGOP exceeds maximum number of BFrames configured".

Cause is for HEVC encode, CO3->GPB is set to "on" by default, which makes GPB frames are used instead of P frames. In CTS test, GPB frames are considered B frames.

Solution is for HEVC encode, set CO3->GPB to "off" in mediasdk_c2 to use P frames not GPB frames. But in onevpl-intel-gpu, HEVC encode caps did not enable "GPB off" before, change HEVC encode caps to enable it.

Tracked-On: OAM-118626

Some CTS tests of android.videocodec.cts.VideoEncoderMaxBFrameTest#
testMaxBFrameSupport failed with "Number of BFrames in a SubGOP
exceeds maximum number of BFrames configured".

Cause is for HEVC encode, CO3->GPB is set to "on" by default, which
makes GPB frames are used instead of P frames. In CTS test, GPB
frames are considered B frames.

Solution is for HEVC encode, set CO3->GPB to "off" in mediasdk_c2
to use P frames not GPB frames. But in onevpl-intel-gpu, HEVC encode
caps did not enable "GPB off" before, change HEVC encode caps to
enable it.

Tracked-On: OAM-118626
Signed-off-by: Lina Sun <[email protected]>
@sysopenci
Copy link

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

1 similar comment
@sysopenci
Copy link

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

SUCCESS: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web for more details.

1 similar comment
@sysopenci
Copy link

SUCCESS: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

Android CI has started MERGE Build for this pr ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci sysopenci merged commit 68bbe35 into projectceladon:main Jun 4, 2024
14 checks passed
@sysopenci
Copy link

Android CI has completed MERGE Build for this pr, build is SUCCESS. Please check the linked Tracked-On issue/Android CI Web for more details. For Binaries: /cactus-absp-or-local/celadon-merge/528

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants