Opened 5 years ago
Closed 5 years ago
#553 closed defect (fixed)
Issue in RowHeight[] access
Reported by: | forayr | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | spec | Version: | VVC D6 vE |
Keywords: | Cc: | ksuehring, bbross, XiangLi, fbossen, jvet@… |
Description
By its construction in 6.5.1, RowHeight is accessed by a tile row index. However, when it is used in table and semantics, it is accessed using a tile index.
The following change should be done in the specification:
In the PPS description:
for( i = 0; brick_splitting_present_flag && i <= num_tiles_in_pic_minus1; i++ ) { + tileY = i / NumTileCols - if( RowHeight[ i ] > 1 ) + if( RowHeight[ tileY ] > 1 ) brick_split_flag[ i ] if( brick_split_flag[ i ] ) { - if( RowHeight[ i ] > 2 ) + if( RowHeight[ tileY ] > 2 ) uniform_brick_spacing_flag[ i ]
In the semantics of brick_height_minus1[i] and num_brick_rows_minus2[i], RowHeight[ i ] should be replaced by RowHeight[ tileY ]
Change history (2)
comment:1 Changed 5 years ago by yjchang
comment:2 Changed 5 years ago by yk
- Resolution set to fixed
- Status changed from new to closed
Note: See TracTickets for help on using tickets.
There are solutions in two contributions, i.e., P0096 and P0433, to address the typos. I think it will be discussed in the meeting.