Opened 9 years ago

Closed 9 years ago

#757 closed defect (fixed) unvalued refIdxCol and listCol used

Reported by: conrad.ho Owned by: bbross
Priority: minor Milestone: D9
Component: Text Version: D9 (K1003) v10
Keywords: Cc: bbross, wjhan, jct-vc@…


The spec said:
"The variables mvLXCol and availableFlagLXCol are derived as follows.

– If one or more of the following conditions are true, both components of mvLXCol are set equal to 0 and availableFlagLXCol is set equal to 0.

–colPb is coded in an intra prediction mode.
-colPb is marked as "unavailable".
–slice_temporal_mvp_enable_flag is equal to 0.
–LongTermRefPic( currPic, refIdxLX, ListX ) is not equal to LongTermRefPic( colPic, refIdxCol, listCol ).

– Otherwise, the motion vector mvCol, the reference index refIdxCol, and the reference list identifier listCol are derived as follows.


However, for the 4th condition, the values of refIdxCol and listCol are specified in "otherwise" part; thus LongTermRefPic(colPic, refIdxCol, listCol) is vague.

Change History (6)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc bbross wjhan jct-vc@… added

comment:2 Changed 9 years ago by conrad.ho

According to HM logic (see the function TComDataCU::xGetColMVP), refIdxCol and listCol are calculated before the comparison of LongTermRefPic( currPic, refIdxLX, ListX ) and LongTermRefPic( colPic, refIdxCol, listCol ).

WD should be modified accordingly, to eliminate the usage of un-valued refIdxCol and listCol.

comment:3 Changed 9 years ago by bbross

  • Milestone changed from D8 to D9
  • Version changed from D8 (K0030) v3 to D9 (K1003) v10

comment:4 Changed 9 years ago by bbross

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

comment:5 Changed 9 years ago by bbross

  • Resolution fixed deleted
  • Status changed from closed to new

comment:6 Changed 9 years ago by bbross

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

This has been fixed in JCTVC-K1003_v11.

Note: See TracTickets for help on using tickets.

This list contains all users that will be notified about changes made to this ticket.

These roles will be notified: Reporter, Owner, Subscriber, Participant

  • Benjamin Bross(Owner, Subscriber, Participant)
  • Conrad Ho(Reporter, Participant)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)
  • Woo-Jin Han(Subscriber)