Opened 9 years ago

Closed 9 years ago

#431 closed defect (fixed)

Missing initial value for candIntraPredModeB and candIntraPredModeA

Reported by: Changcai Lai Owned by: bbross
Priority: minor Milestone: WD6
Component: Text Version: D6 (H1003) dI/dJ/dK
Keywords: Cc: bbross, wjhan, jct-vc@…


(actual version dK)

  1. The candModeList[ x ] with x=0..2 is derived as follows:

– If candIntraPredModeB is equal to candIntraPredModeA, the following applies:

candIntraPredModeB and candIntraPredModeA have no initial values, suppose candIntraPredModeN is equal to intraPredModeN but need to be described in text.

Change History (4)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc bbross wjhan jct-vc@… added

comment:2 Changed 9 years ago by tktan

Why do we need both candIntraPredModeN and intraPredModeN

Why don't we just replace "intraPredMode" with "candIntraPredMode" in the subclause 4 above.

Looking a little further down to subclause 6. 1) i., ii. and iii.
Why don't we just describe this step as

1) The elements of the list candModeList are sorted such that candModeList[x] < candModeList[x+1], for x=0 to 1.

In the second clause 6. 2) steps ii. to iv. are just a repeat of the same action.
It is better to replace it with a single “for loop”.

2) IntraPredMode[xB][yB] is derived as the following ordered steps:

  1. IntraPredMode[ xB ][ yB ] = rem_intra_luma_pred_mode
  2. for (i=0; i<3; i++)

if IntraPredMode[ xB ][ yB ] is greater than or equal to candModeList[ i ], the value of IntraPredMode[ xB ][ yB ] is increased by one

comment:3 Changed 9 years ago by tktan

Also it would seem that "prev_intra_pred_flag[ xB ][ yB ]" in Step 6 of Subclause 8.4.1 has already been changed to "prev_intra_luma_pred_flag[ xB ][ yB ]".

comment:4 Changed 9 years ago by bbross

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

Fixed in JCTVC-I1003_d9.

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(Owner, Subscriber, Participant)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)
  • TK Tan(Participant)
  • Woo-Jin Han(Subscriber)