Opened 4 years ago

Closed 4 years ago

#1196 closed defect (fixed)

Typo in variable names

Reported by: bheng Owned by:
Priority: minor Milestone:
Component: spec Version: VVC D10 vA
Keywords: Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…

Description

The position (xTbComp, yTbComp) is used in several places but never defined anywhere. I believe these should either be ( xTbCmp, yTbCmp ) or ( xCurr, yCurr ) depending on the context.

On a related note, several of the input names mentioned for 8.7.5.1 (just invoked as 8.7.5 in the text) don't match up with the names used in the actual 8.7.5.1 subclause.

( xCurr, yCurr ), nCurrSw, and nCurrSh

versus

( xTbComp, yTbComp ), nTbW, and nTbH

versus

( xB, yB ), bWidth, and bHeight

Change history (2)

comment:1 Changed 4 years ago by jlchen

Thanks for reporting the bug. Those (xTbComp, yTbComp)(s) will be changed to ) (xTbCmp, yTbCmp)(s) in vE release

Regarding the input parameter names not matching up with the names inside the subclause, we will check whether this exist in many places or only a few places. And decide whether need to fix them or not.

comment:2 Changed 4 years ago by bbross

  • Resolution set to fixed
  • Status changed from new to closed

Good catch, thanks! Fixed the input parameters and the cross-reference to 8.7.5.1 (instead of the parent clause 8.7.5) in JVET-S2001-vE.

Note: See TracTickets for help on using tickets.