Opened 4 years ago

Closed 4 years ago

#1314 closed defect (fixed)

Unnecessary variables in Section 8.5.2.10

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

Description

The following sentence seems unnecessary since currCB and currPic are not used elsewhere in this section:

The variable currCb specifies the current luma coding block at luma location ( xCb, yCb ) and the variable currPic specifies the current picture.

Equations 581 and 583 also seem unnecessary, since refIdxB is not an output of this subclause and is used elsewhere.

refIdxB = RefIdxLX[ xNbBk ][ yNbBk ] (581)

refIdxB = RefIdxLY[ xNbBk ][ yNbBk ] (583)

Change history (3)

comment:1 Changed 4 years ago by jlchen

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

Good catch, Brian! Those text look redundant to me. Will be fixed in S2001-vH release.

comment:2 Changed 4 years ago by bheng

  • Resolution fixed deleted
  • Status changed from closed to reopened

Thanks for all the fixes!

It looks like this sentence is still in the draft:

"The variable currCb specifies the current luma coding block at luma location ( xCb, yCb ) and the variable currPic specifies the current picture."

Probably you could remove it unless you've noticed something I missed.

comment:3 Changed 4 years ago by bbross

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

Thanks for checking and reopening! The unnecessary sentence will be removed in JVET-S2001-vH.

Note: See TracTickets for help on using tickets.