Skip to content

Misuse of pMiCol/RowStarts of StdVideoAV1TileInfo. #152

@zzoon

Description

@zzoon

The current implementation in VulkanAV1Decoder::DecodeTileInfo in vk_video_decoder/libs/NvVideoParser/src/VulkanAV1Decoder.cpp

pic_data->MiColStarts[i] = start_sb;

which means it considers SB as a unit of pMiColStarts instead of a MI unit
IIUC, it should be a MI unit.

The spec says:

pMiColStarts is a pointer to an array of TileCols number of unsigned integers 
that corresponds to MiColStarts defined in section 6.8.14 of the [AV1 Specification]

pMiRowStarts is a pointer to an array of TileRows number of unsigned integers
that corresponds to MiRowStarts defined in section 6.8.14 of the [AV1 Specification]

And AV1 spec is here: https://aomediacodec.github.io/av1-spec/av1-spec.pdf

Thoughts?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions