Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#60 closed defect (fixed)

Missing split for the case TU_MAX_TR_SPLIT in IntraSearch::xEncCoeffQT

Reported by: tpoirier Owned by:
Priority: minor Milestone:
Component: BMS Version: BMS-1.1
Keywords: Cc: ksuehring, XiangLi, fbossen, jvet@…

Description

In xEncCoeffQT function, that estimates the cost of intra residual, the loop over inferred TU split for large blocks is missing. This leads to a wrong cost estimation for 128x128 Intra blocks.
CTC are not impacted since LCTUFast parameter, that removes the testing of 128x128 Intra CU, is enabled.
The following patch solve this issue.

Attachments (1)

bugfix_MAX_TR_SPLIT_Intra.patch (935 bytes) - added by tpoirier 3 years ago.

Download all attachments as: .zip

Change history (3)

Changed 3 years ago by tpoirier

comment:1 Changed 3 years ago by XiangLi

  • Milestone set to BMS-1.2
  • Resolution set to fixed
  • Status changed from new to closed

Fixed in r1374 (also fixed in VTM).

comment:2 Changed 3 years ago by ksuehring

  • Milestone BMS-1.2 deleted

Milestone deleted

Note: See TracTickets for help on using tickets.