Incorrect size and reset for m_weightPredTable

[audrey.turquin]

Since the introduction of the syntax elements num_l0_weights and num_l1_weights in the PH, the size of m_weightPredTable is not correct anymore.

According to the specification text, num_l0_weights and num_l1_weights shall be in the range 0 to num_ref_entries.
num_ref_entries can be up to MaxDpbSize + 13. So, at the maximum num_ref_entries can be equal to 16 + 13 = 29.

However, in the VTM, the size of m_weightPredTable is limited to MAX_NUM_REF = 16, which is not enough.
The size of m_weightPredTable should be increased to 29.

Moreover, a reset of m_weightPredTable at picture header level is missing.

[ksuehring]

It seems like

​https://vcgit.hhi.fraunhofer.de/jvet/VVCSoftware_VTM/merge_requests/1237

would fix this issue as well?

[audrey.turquin]

This is indeed related to this issue.
However in the size of m_weightPredTable, this is MAX_NUM_REF which is used, and the MR you pointed out changes MAX_NUM_REF_PICS. So the MR would not fix this issue, unless we make the change to use MAX_NUM_REF_PICS to set the size of m_weightPredTable.

I wonder if there is some confusion in using MAX_NUM_REF or MAX_NUM_REF_PICS in the VTM as the use doesn't match the comment (but it is seemless as both constants have the same value for the moment).
Maybe one should be renamed to have a 'MAX_NUM_REF_ENTRIES' (=29) which is more clear ?

[audrey.turquin]

Corresponding MR:
​https://vcgit.hhi.fraunhofer.de/jvet/VVCSoftware_VTM/merge_requests/1407

[audrey.turquin]

The spec has been changed to do not allow values greater than 15.
(see ticket https://jvet.hhi.fraunhofer.de/trac/vvc/ticket/908).
This solves the issue and the VTM no longer needs to be fixed.