Opened 10 years ago

Closed 6 years ago

#1398 closed defect (fixed)

Typos in the Text

Reported by: CFeldmann Owned by:
Priority: minor Milestone: v5
Component: Text Version: v4 (12/2016)
Keywords: Cc: bbross, wjhan, jct-vc@…

Description

In ITU-T H.265 (10/2014) as well as JCTVC-R1013_v6 I found some typos:

In F.7.3.2.3.3:
"if( resample_phase_set_prsent_flag[ i ] ) {" sould be "if( resample_phase_set_present_flag[ i ] ) {"

In F.7.3.2.3.5:
"idxShiftY = idxY + ( ( i << ( cm_octant_depth − inpDepth ) )". There is one too many open brackets "(".

In H.8.1.4.2:
... "where xPb = xB << 4 and xPb = xB << 4, for" ...
I guess the correct version is:
... "where xPb = xB << 4 and yPb = yB << 4, for" ...

Also in H.8.1.4.2:
In steps 4 and 5 the index "xP" and "yP" are used multiple times. However, these indices were not defined in this context. I guess the correct indices are "xPb" and "yPb".

Change History (3)

comment:1 Changed 10 years ago by DefaultCC Plugin

  • Cc bbross wjhan jct-vc@… added

comment:2 Changed 7 years ago by bbross

  • Milestone set to v5
  • Version set to v4 (12/2016)
  1. The typo ‘resample_phase_set_prsent_flag’ is fixed in the 2016/12 release of the HEVC specification.
  2. Agreed by the authors of the AHG2 report JCTVC-AD0002. The redundant ‘(‘ in subclause F.7.3.2.3.6 is still present in the 2016/12 release of the HEVC specification.

Fix:
In F.7.3.2.3.6, change

idxShiftY = idxY + ( ( i << ( cm_octant_depth − inpDepth ) )

to

idxShiftY = idxY + ( i << ( cm_octant_depth − inpDepth ) )

  1. The issue regarding xPb/yPb is fixed in the 2016/12 release of the HEVC specification (note: now affected clause is H.8.1.4.3 and the variables were renamed to xP/yP).
  2. xP/yP are now the correct names to use (so no action needed).

comment:3 Changed 6 years ago by bbross

  • Resolution set to fixed
  • Status changed from new to closed

fixed in v5 (2018/02)

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

  • Benjamin Bross(Subscriber, Participant)
  • Christian Feldmann(Reporter)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)
  • Woo-Jin Han(Subscriber)