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

There are solutions in two contributions, i.e., P0096 and P0433, to address the typos. I think it will be discussed in the meeting.

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.