Eyevinn / hi264

88%

LAST BUILD ON BRANCH main

branch: SELECT

CHANGE BRANCH

x

• No branch selected
• add-pre-commit-install
• chore/remove-goreportcard-badge
• ci/node24-actions
• color-space
• default-no-deblock
• expand-docs
• extend-example
• feat/bg-anim
• feat/continue-poc
• feat/go-v1-24
• feat/grid-8x8
• feat/image-bg
• feat/pic-timing-sei
• fix-badges
• fix/cavlc-trailing-ones-sign
• fix/signal-level
• main
• optimize
• optimize-write
• overlay-text
• ver-0-10-0
• ver0.9.0

Build # 32488580255

Build Type

push

github

Committed by tobbee

Commit Message

test: require CAVLC and CABAC to decode identically

Both entropy coders serialize the same quantized coefficients, so their
decoded frames must match bit for bit. Encode detailed PlaneGrid content
(every block differs from its neighbours, so all three I_16x16 prediction
modes get used and mixed-sign trailing-one pairs actually occur) with both
coders across 16x16/8x8 granularity, QP 12-42, and deblocking on and off.

This pins the invariant the reversed trailing_ones_sign_flag broke, from
the other direction: CABAC emits each sign next to its own level so it was
never affected, and now a regression in either coder fails the test.
Reintroducing the sign-order bug fails 20 of the 20 subtests.

Coverage Stats

8161 of 9300 relevant lines covered (87.75%)

1.01 hits per line