Opened 5 years ago
Closed 5 years ago
#957 closed task (fixed)
Factor SubWidthC used with scaling_win_top_offset?
Reported by: | vdrugeon | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | spec | Version: | |
Keywords: | Cc: | ksuehring, bbross, XiangLi, fbossen, jvet@… |
Description
In the PPS semantics, the variable PicOutputHeightL is specified as follows: "PicOutputHeightL = pic_height_in_luma_samples − SubWidthC * ( scaling_win_bottom_offset + scaling_win_top_offset )" At first sight, SubHeightC would seem to make more sense than SubWidthC in that equation. Can an expert please check?
Similarly, in 8.5.6.3.1 General: "The variable fRefTopOffset is set equal to ( ( SubWidthC * scaling_win_top_offset ) << 10 ), where scaling_win_top_offset is the scaling_win_top_offset for the reference picture." Is SubWidthC correct here, or should it be SubHeightC instead? Can an expert please check?
Same question for the following equation: "refySbL = ( ( ( ySb − ( SubWidthC * scaling_win_top_offset ) ) << 4 ) + refMvLX[ 1 ] ) * scalingRatio[ 1 ]".
Again same question for the following equation: "refySbC = ( ( ( ySb − ( SubWidthC * scaling_win_top_offset ) ) / SubHeightC << 5 ) + refMvLX[ 1 ] ) * scalingRatio[ 1 ] + addY".
Change history (2)
comment:1 Changed 5 years ago by bheng
comment:2 Changed 5 years ago by yk
- Resolution set to fixed
- Status changed from new to closed
Good catches! Thanks! Will be fixed in R2001-v8.
Yes, I agree. The scale factor for bottom/top offsets should be changed to SubHeightC.
I believe those are correct in the corresponding JVET-Q0487 proposal, for reference.