Opened 4 years ago

Closed 4 years ago

#908 closed defect (fixed)

Maximum number of PH wpred weights

Reported by: bheng Owned by:
Priority: minor Milestone:
Component: spec Version: VVC D8 vB
Keywords: Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…

Description

Should the range of num_lx_weights in the weighted prediction parameters semantics be limited to a max of 15?

The current range is as follows:

"The value of num_l0_weights shall be in the range of 0 to num_ref_entries[ 0 ][ RplsIdx[ 0 ] ], inclusive."

I understand the reason to allow num_ref_entries to be as large as 29. However, only the first 15 of those can ever be active (num_ref_idx_default_active <= 15), so as far as I can tell, weight 16 to 29 could never be used.

Suggest changing the range to:

"The value of num_l0_weights shall be in the range of 0 to MIN( 15, num_ref_entries[ 0 ][ RplsIdx[ 0 ] ] ), inclusive."

Change history (3)

comment:1 Changed 4 years ago by yk

Thanks! Your suggestion makes sense to me. Unless there is a different opinion, in which case we should have more discussions, I will make this change when I take the token next time.

comment:2 Changed 4 years ago by yk

Fixed in Q2001-vE (for both num_l0_weights and num_l1_weights).

comment:3 Changed 4 years ago by yk

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.