Opened 12 years ago

Closed 12 years ago

# Missing initial value for candIntraPredModeB and candIntraPredModeA

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

### Description

(actual version dK)
8.4.1
......

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.

### comment:1 Changed 12 years ago by DefaultCC Plugin

• Cc bbross wjhan jct-vc@… added

### comment:2 Changed 12 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 12 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 12 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)