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
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.
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.