Opened 4 years ago

Closed 4 years ago

#1351 closed defect (fixed)

Uninitialized predFlagLXConstK in Section 8.5.5.6

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

Description

In Section 8.5.5.6, predFlagLXConstK are never set to 0. They are either set to 1 or undefined.

Ideally, it would be a good idea to specify predFlagLXConstK = 0, refIdxLXConstK = -1, and cpMvLXConstK = (0,0) for all the availableFlagLX equal to FALSE conditions so that there aren't undefined reference index and motion vectors floating around. Otherwise conditions elsewhere such as "... have the same motion vectors and the same reference indices" aren't strictly defined.

Change history (5)

comment:1 Changed 4 years ago by jlchen

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

Thanks for reporting the issue. The following initialization will be added to section 8.5.5.6.

The availability flags of the constructed affine control point motion vector merging candidates availableFlagConstK, with K = 1..6, are initialized to be equal to FALSE.

For the suggestion of setting "refIdxLXConstK = -1, and cpMvLXConstK = (0,0) for all the availableFlagLX equal to FALSE", I tend to NOT doing it since many places need to be considered and it's easy to overlooked.

comment:2 Changed 4 years ago by bheng

  • Resolution fixed deleted
  • Status changed from closed to reopened

The text to initialize availableFlagConstK was the fix for Ticket #1293. But, I don't think it solves the problem for initializing predFlagL0ConstK = 0 and predFlagL1ConstK = 0, in case an available candidate doesn't use List0 or List1 prediction.

comment:3 Changed 4 years ago by jlchen

Yes, you're right. Would the following initialization process of the predFlagLXConstK solve the problem?

The prediction list utilization flags predFlagLXConstK, with with K = 1..6, X = 0..1, are initialized to be equal to FALSE.

comment:4 Changed 4 years ago by bheng

Yes, that would work fine. Thanks!

comment:5 Changed 4 years ago by jlchen

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

Thanks for comfirmation.

Note: See TracTickets for help on using tickets.