Opened 5 years ago

Closed 5 years ago

#670 closed defect (fixed)

JVET-P0158: Apply ALF VB on the bottom CTU row

Reported by: bheng Owned by:
Priority: minor Milestone:
Component: VTM Version: VTM-7.0rc1
Keywords: Cc: ksuehring, XiangLi, fbossen, jvet@…

Description

There is one additional modification needed to treat the final CTU row the same as all other CTU rows for VB processing.

Specifically, in AdaptiveLoopFilter::deriveClassification():

The following code

( ( i - blk.pos().y + blkDst.pos().y + nHeight >= m_picHeight ) ? m_picHeight : m_alfVBLumaPos )

should just be

m_alfVBLumaPos

Change history (2)

comment:1 Changed 5 years ago by jennylai

yes, you are right. This part is missing.

Thanks for reporting.

comment:2 Changed 5 years ago by bheng

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

Fixed in merge request jvet/VVCSoftware_VTM!1167

Note: See TracTickets for help on using tickets.