Opened 5 years ago
Closed 4 years ago
#1098 closed defect (fixed)
bug in DMVR bounding box calculation
Reported by: | lenchik | Owned by: | |
---|---|---|---|
Priority: | trivial | Milestone: | |
Component: | spec | Version: | VVC D8 vB |
Keywords: | Cc: | chenhuanbang@…, ksuehring, bbross, XiangLi, fbossen, jvet@… |
Description
there seems to be a bug in the spec related to the DMVR memory bounding box. one additional sample seems to be fetched at the right and bottom of the box.
for a 16x16 luma subblock, the size of the bounding box should be 23x23, but in the current VVC spec it seems to be 24x24. the fix is below, the software seems to work as intended.
current VVC spec:
– When dmvrFlag is equal to 1, the following applies:
xInti = Clip3( xSbIntL − 3, xSbIntL + sbWidth + 4, xInti ) (957)
yInti = Clip3( ySbIntL − 3, ySbIntL + sbHeight + 4, yInti ) (958)
should be:
– When dmvrFlag is equal to 1, the following applies:
xInti = Clip3( xSbIntL − 3, xSbIntL + sbWidth + 3, xInti ) (957)
yInti = Clip3( ySbIntL − 3, ySbIntL + sbHeight + 3, yInti ) (958)
And similarly for chroma interpolation filtering, if the subblock size is 8x8, the size of bounding box should be 11x11.
current VVC spec:
– When dmvrFlag is equal to 1, the following applies:
xInti = Clip3( xSbIntC − 1, xSbIntC + sbWidth + 2, xInti ) (975)
yInti = Clip3( ySbIntC − 1, ySbIntC + sbHeight + 2, yInti ) (976)
should be:
– When dmvrFlag is equal to 1, the following applies:
xInti = Clip3( xSbIntC − 1, xSbIntC + sbWidth + 1, xInti ) (975)
yInti = Clip3( ySbIntC − 1, ySbIntC + sbHeight + 1, yInti ) (976)
Change history (2)
comment:1 Changed 4 years ago by jlchen
comment:2 Changed 4 years ago by bbross
- Resolution set to fixed
- Status changed from new to closed
Good catch, thanks!
This will be fixed in first verison of D10.
Thanks for pointing this out. You're correct.
The intention is to clap the position within 4 luma samples to the right board of luma block as follows:
xInti = Clip3( xSbIntL − 3, xSbIntL + sbWidth - 1 + 4, xInti ) (957)
yInti = Clip3( ySbIntL − 3, ySbIntL + sbHeight - 1 + 4, yInti ) (958)
Therefore the four equations should be fixed as suggested.