Skip to content

Write tile sizes into the frame header by default - #238

Merged
cpncf merged 2 commits into
mainfrom
add_tile_size_in_fh
Aug 6, 2026
Merged

Write tile sizes into the frame header by default#238
cpncf merged 2 commits into
mainfrom
add_tile_size_in_fh

Conversation

@kpchoi

@kpchoi kpchoi commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator

This enables the tile_size_present_in_fh_flag path discussed in #228, so a
decoder can index individual tiles from the frame header without scanning the
tile data.

  • The encoder now writes tile_size_in_fh[] (u(32) per tile) by default,
    reusing the existing frame header rewrite after tile encoding, in the same
    way pbu_size is backfilled
  • oapve_config() gets OAPV_CFG_SET/GET_TILE_SIZE_IN_FH, settable per
    frame through OAPV_CFG_FRM(); the encoder app disables it with
    --disable-tile-size-in-fh
  • The per-frame RC slot indexing in oapve_encode() now uses the loop index
    directly instead of a clamped copy, since the frame count is already
    validated at the function entry

kpchoi added 2 commits August 6, 2026 20:33
Signed-off-by: KP Choi <kp5.choi@samsung.com>
Signed-off-by: KP Choi <kp5.choi@samsung.com>

@cpncf cpncf left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cpncf
cpncf merged commit 6be26ed into main Aug 6, 2026
7 checks passed
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.

2 participants