Opened 5 years ago
Closed 5 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 5 years ago by yk
comment:2 Changed 5 years ago by yk
Fixed in Q2001-vE (for both num_l0_weights and num_l1_weights).
comment:3 Changed 5 years ago by yk
- Resolution set to fixed
- Status changed from new to closed
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.