Opened 8 years ago

Closed 4 years ago

#1441 closed defect (fixed)


Reported by: adil.parvez Owned by:
Priority: minor Milestone:
Component: SCC text Version:
Keywords: typos JCTVC-V1005-v1 Cc: rajan_joshi, jct-vc@…


In Transform unit syntax.
tu_residual_act_flag ae(v)

In Transform unit semantics.
tu_residual_act_flag[x0][y0] ...

Syntax table is missing array subscripts.

In palette syntax and semantics, travScan[·][·] is used, this should be ScanOrder[log2BlockSize][3][·][·], also ScanOrder in the semantics should be updated to include travScan.

In General, repeated condition

  • nPbSw is equal to 8


  • noIntegerMV is equal to 1

should be

  • noIntegerMv is equal to 1

In equation (8-108) the variables xCurr, yCurr are used, but aren't defined, it probably means xCb, yCb.

In 8.4.2 Derivation process for luma intra prediction mode, candModeList[mpm_idx], should be candModeList[mpm_idx[xPb][yPb]].

Change History (3)

comment:1 Changed 8 years ago by adil.parvez

Also the log2BlockSize subscript of ScanOrder should be log2(nCbs) or log2CbSize passed through from coding_unit.

comment:2 Changed 4 years ago by crosewarne

In JCTVC-W1005:

  • tu_residual_act_flag missing array indices is fixed.
  • Palette syntax uses ScanOrder and log2BlockSize is defined as Log2(nCbS).
  • In, duplicated nPbSw condition is removed and incorrect capitalization is fixed.
  • mpm_idx for candModeList has array indices included.

Thus, all raised aspects are addressed.

comment:3 Changed 4 years ago by crosewarne

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.

This list contains all users that will be notified about changes made to this ticket.

These roles will be notified: Reporter, Owner, Subscriber, Participant

  • Adil Parvez(Reporter, Participant)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)
  • Rajan Joshi(Subscriber)