id summary reporter owner description type status priority milestone component version resolution keywords cc 606 Weighted prediction process needs major revision bbross bbross "The following issues are present in the WP process 8.5.2.2.3: 1. The input ""bit-depth of the chroma component, bitDepth"" is wrong for luma invocation. 2. Variables shift1, shift2, offset1 and offset2 are derived in 8.5.2.2.3 but used in 8.5.2.2.3.1 Default weighted sample prediction process 3. The whole process is inconsistent w.r.t. luma/chroma in 8.5.2.2.3.2 Weighted sample prediction process: - there is a ""with H being replaced by Y for luma samples and by C for chroma samples"". How do we know that is is luma or chroma? - For the derivation of logWDC, o0C, o1C, and w0C, w1C, ""If C is equal to L""... Where comes C from? - Why is BitDepthC and BitDepthY used in this derivation although corresponding bitDepth is input parameter and set to the global variables when invoked? - Wouldn't it be better to pass a cIdx and derive variables depending on that variable? 4. logWDC should be log2WDC and derivation of these variable should be moved at the beginning (see #433) 5. Invocation of the WP process for chroma should be split into two paragraphs as it is done for other processes. 6. Minor improvement: Since it is not used somewhere else, ""the same as specified in sub clause 8.5.2.2.3"" should be replaced by the actual inputs and outputs. (see #433) Who is going to fix this?" defect closed major D7 Text D7 (I1003) d5 fixed bbross wjhan jct-vc@…