Opened 8 years ago

Closed 4 years ago

#1441 closed defect (fixed)

Typos.

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

Description


In 7.3.8.10 Transform unit syntax.
...
tu_residual_act_flag ae(v)
...

In 7.4.9.10 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 8.5.2.3.1 General, repeated condition

  • nPbSw is equal to 8

Also

  • 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 8.5.3.2.1, 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)