Skip to content

Reuse local RDOQ rate costs and entropy flags - #989

Open
xiujunwang-dev wants to merge 1 commit into
Multicorewareinc:masterfrom
xiujunwang-dev:rdoq-entropy-optimization
Open

xiujunwang-dev wants to merge 1 commit into
Multicorewareinc:masterfrom
xiujunwang-dev:rdoq-entropy-optimization

Conversation

@xiujunwang-dev

Copy link
Copy Markdown
Contributor

Cache the coefficient-group scan base and zero-significance cost, and pass the used level-absolute bit cost directly to the local rate helpers. Reuse CBF queries within transform and coefficient coding. The aim is to reduce redundant local work while preserving coding decisions.

Test results against unmodified master f2cf32d on macOS arm64 with AppleClang 21:

  • 8-bit Release: 8 cases of 240 frames; all bitstreams match the baseline.
  • 10-bit Release: 4 cases of 240 frames; all bitstreams match the baseline.
  • 8-bit CHECKED_BUILD=ON: 2 cases of 240 frames; bitstreams match and no assertions fail.

All 14 comparisons pass byte for byte. Synthetic inputs cover RDOQ 1/2, 4:2:0/4:2:2/4:4:4, 8-bit monochrome, intra/inter coding, QP/CRF, psy-RDOQ and sign hiding. Checked-build cases cover 4:2:0 and 4:2:2. Build-info SEI is disabled with --no-info for bitstream comparisons.

Cache the coefficient-group scan base and zero-significance cost, and
pass the used level-absolute bit cost directly to the local rate helpers.
Reuse CBF queries within transform and coefficient coding.
The aim is to reduce redundant local work while preserving coding
decisions.

Test results against unmodified master f2cf32d on macOS arm64 with
AppleClang 21:

- 8-bit Release: 8 cases of 240 frames; all bitstreams match the baseline.
- 10-bit Release: 4 cases of 240 frames; all bitstreams match the baseline.
- 8-bit CHECKED_BUILD=ON: 2 cases of 240 frames; bitstreams match and no
  assertions fail.

All 14 comparisons pass byte for byte. Synthetic inputs cover RDOQ 1/2,
4:2:0/4:2:2/4:4:4, 8-bit monochrome, intra/inter coding, QP/CRF,
psy-RDOQ and sign hiding. Checked-build cases cover 4:2:0 and 4:2:2.
Build-info SEI is disabled with --no-info for bitstream comparisons.
@kirithika7
kirithika7 requested a review from mcw-Lavanya October 8, 2026 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant