Opened 4 years ago

Closed 4 years ago

#1120 closed defect (invalid)

Fix condition for TMVP

Reported by: pierrick.bouvier Owned by:
Priority: minor Milestone: VTM-9.1
Component: VTM Version:
Keywords: Cc: ksuehring, XiangLi, fbossen, jvet@…

Description

In function PU::getInterMergeCandidates (UnitTools.cpp),

in part related to 8.5.2.11 Derivation process for temporal luma motion vector prediction:
If ph_temporal_mvp_enabled_flag is equal to 0 or
(cbWidth*cbHeight) is less than or equal to 32,
both components of mvLXCol are set equal to 0 and availableFlagLXCol is set equal to 0

In VTM implementation, 12 is present instead of 32 (probably a typo).

MR: https://vcgit.hhi.fraunhofer.de/jvet/VVCSoftware_VTM/-/merge_requests/1688

Change history (2)

comment:1 Changed 4 years ago by pierrick.bouvier

I missed the fact that VTM checks this with an addition instead of multiplication.

cbWidth + cbHeight > 12, instead of cbWidth * cbHeight > 32.

Both conditions are equivalent.

Thus, this ticket can be closed.

comment:2 Changed 4 years ago by pierrick.bouvier

  • Resolution set to invalid
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.