Opened 10 years ago

Closed 10 years ago

#289 closed defect (fixed)

Text bug related to Temporal MV prediction (8.4.2.1.8)

Reported by: iperroux Owned by: bbross
Priority: minor Milestone:
Component: Text Version: WD5 (G1103) d4
Keywords: Cc: bbross, wjhan, jct-vc@…

Description

[Reference text taken from WD5-d5]
In step 6 of 8.4.2.1.8 (Derivation process for temporal luma motion vector prediction), the value of N is not correctly defined in “with N being value of RefIdxLX”.

It was suggested by the editor team to replace the following paragraph:

  • If all PicOrderCnts of pictures in all reference picture lists are smaller than PicOrderCnt( currPic ), the motion vector mvCol, the reference index refIdxCol and ListCol are set equal to MvLN[ xPCol ][ yPCol ], RefIdxLN[ xpCol ][ ypCol ] and LN, respectively, with N being value of RefIdxLX.
  • Otherwise (one of PicOrderCnt of pictures in all reference picture lists are larger than PicOrderCnt( currPic ) ), the motion vector mvCol, the reference index refIdxCol and ListCol are set equal to MvLN[ xPCol ][ yPCol ], RefIdxLN[ xpCol ][ ypCol ] and LN, respectively, with N being value of collocated_from_l0_flag.

by this one:

  • If PicOrderCnt( pic ) of every picture pic in every reference picture list is less than or equal to PicOrderCnt( currPic ), the motion vector mvCol, the reference index refIdxCol and ListCol are set equal to MvLX[ xPCol ][ yPCol ], RefIdxColLX and LX, respectively, with X being the value of X this process is invoked for.
  • Otherwise (PicOrderCnt( pic ) of at least one picture pic in at least one reference picture list is greater than PicOrderCnt( currPic )), the motion vector mvCol, the reference index refIdxCol and ListCol are set equal to MvLN[ xPCol ][ yPCol ], RefIdxColLN and LN, respectively, with N being the value of collocated_from_l0_flag.

Attachments (1)

fix289.docx (144.5 KB) - added by bbross 10 years ago.
Based on WD5 (G1103) d5

Download all attachments as: .zip

Change History (6)

comment:1 Changed 10 years ago by DefaultCC Plugin

  • Cc bbross wjhan jct-vc@… added

comment:2 Changed 10 years ago by iperroux

Actually, if including the WD5_d5 updates about RefIdxColLN, the new text should be:

•If PicOrderCnt( pic ) of every picture pic in every reference picture list is less than or equal to PicOrderCnt( currPic ), the motion vector mvCol, the reference index refIdxCol and ListCol are set equal to MvLX[ xPCol ][ yPCol ], RefIdxLX[ xPCol ][ yPCol ] and LX, respectively, with X being the value of X this process is invoked for.
•Otherwise (PicOrderCnt( pic ) of at least one picture pic in at least one reference picture list is greater than PicOrderCnt( currPic )), the motion vector mvCol, the reference index refIdxCol and ListCol are set equal to MvLN[ xPCol ][ yPCol ], RefIdxLN[ xPCol ][ yPCol ]and LN, respectively, with N being the value of collocated_from_l0_flag.

comment:3 Changed 10 years ago by bbross

Another issue in that process is that "ListCol" is not always defined. All this will be fixed in the next release of the draft.

Changed 10 years ago by bbross

Based on WD5 (G1103) d5

comment:4 Changed 10 years ago by bbross

The indentation and the enumeration of the last part is also broken. All this is fixed in the attached fix267.docx.

Ticket will be closed when incorporated in the next WD version.

comment:5 Changed 10 years ago by bbross

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

Fixed in WD5 (G1103) d6.

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)
  • Isabelle Perroux(Reporter, Participant)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)
  • Woo-Jin Han(Subscriber)