Opened 9 years ago

Closed 6 years ago

#1424 closed defect (fixed)

nCS1L is not defined but used

Reported by: jk73.kim Owned by:
Priority: minor Milestone: v5
Component: RExt text Version: v4 (12/2016)
Keywords: Cc: davidf, joel, teruhiko, jct-vc@…

Description

In "8.5.2 Inter prediction process" (pp.130 in H.265-201504),

– Otherwise, if PartMode is equal to PART_nRx2N, ...

  1. ... ( xBl, yBl ) set equal to ( nCS1L + ( nCbSL >> 2 ), 0 ),

Becasue nCS1L is not defined but used,
it should be changed to "( nCbSL >> 1 )".

Thanks.

Change History (2)

comment:1 Changed 7 years ago by bbross

  • Milestone set to v5
  • Version changed from RExt D7 (Q1005) v8 to v4 (12/2016)

Agreed by the authors of the AHG2 report JCTVC-AD0002 (checked 2016/12 release of the HEVC specification).
Fix:
In 8.5.2, change
the luma location ( xBl, yBl ) set equal to ( nCS1L + ( nCbSL >> 2 ), 0 )
to
the luma location ( xBl, yBl ) set equal to ( ( nCbSL >> 1 ) + ( nCbSL >> 2 ), 0 )

comment:2 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(Participant)
  • David Flynn(Subscriber)
  • jct-vc@…(Subscriber)
  • JK Kim(Reporter)
  • Joel(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)
  • Teruhiko Suzuki(Subscriber)