Opened 5 years ago
Closed 5 years ago
#934 closed task (fixed)
Potential issue in determining vertical luma interpolation filter coefficients
Reported by: | vdrugeon | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | spec | Version: | |
Keywords: | Cc: | ksuehring, bbross, XiangLi, fbossen, jvet@… |
Description
In 8.5.6.3.2 Luma sample interpolation filtering process, when calculating vertical luma interpolation filter coefficients:
"The vertical luma interpolation filter coefficients fLV[ p ] for each 1/16 fractional sample position p equal to yFracL are derived as follows:
– If MotionModelIdc[ xSb ][ ySb ] is greater than 0, and sbWidth and sbHeight are both equal to 4, and scalingRatio[ 0 ] is greater than 28672, the luma interpolation filter coefficients fLV[ p ] are specified in Table 31.
– Otherwise, if MotionModelIdc[ xSb ][ ySb ] is greater than 0, and sbWidth and sbHeight are both equal to 4, and scalingRatio[ 0 ] is greater than 20480, the luma interpolation filter coefficients fLV[ p ] are specified in Table 32.
– Otherwise, if MotionModelIdc[ xSb ][ ySb ] is greater than 0, and sbWidth and sbHeight are both equal to 4, the luma interpolation filter coefficients fLV[ p ] are specified in Table 30.
– Otherwise, if scalingRatio[ 1 ] is greater than 28672, luma interpolation filter coefficients fLV[ p ] are specified in Table 28.
– Otherwise, if scalingRatio[ 1 ] is greater than 20480, luma interpolation filter coefficients fLV[ p ] are specified in Table 29.
– Otherwise, the luma interpolation filter coefficients fLV[ p ] are specified in Table 27 depending on hpelIfIdx set equal to hpelVerIfIdx."
Is it intended that scalingRatio[0], i.e. the horizontal scaling factor, is used for determining the vertical filter coefficients in the first two bullets, but scalingRatio[1], i.e. the vertical scaling factor, is used in subsequent bullets? Could someone familiar with this process please check?
Change history (1)
comment:1 Changed 5 years ago by jlchen
- Resolution set to fixed
- Status changed from new to closed
They should be scalingRatio[1]. will be fixed in vE release.