Changes between Initial Version and Version 3 of Ticket #1230


Ignore:
Timestamp:
Feb 17, 2014, 6:52:49 AM (8 years ago)
Author:
davidf
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #1230

    • Property Cc davidf karlsharman jct-vc@… added
  • Ticket #1230 – Description

    initial v3  
    33there is a possible mismatch in the RExt D5 (01005v4) and HM-12.1+RExt-5.1 for deblocking with 422 subsampling. In the following luma CB/TB partitioning for a 16x16 CB two horizontal edges are considered for filtering:
    44
     5{{{
    56      luma 16x16               
    67-----------------------    <- considered for filtering (1)
     
    1516|          |          |
    1617-----------------------
     18}}}
    1719
    1820For the corresponding chroma part for both 420 and 422 only (1) is considered and (2) is not considered:
    1921
    2022TComLoopFilter.cpp:223
    21 
    22 if ( chFmt!=CHROMA_400 && (bAlwaysDoChroma || (uiPelsInPart>DEBLOCK_SMALLEST_BLOCK) || (iEdge % ( (DEBLOCK_SMALLEST_BLOCK<<'''1''')/uiPelsInPart ) ) == 0 ) )   
    23 {
     23{{{
     24if ( chFmt!=CHROMA_400
     25 && (bAlwaysDoChroma
     26  || (uiPelsInPart>DEBLOCK_SMALLEST_BLOCK)
     27  || (iEdge % ( (DEBLOCK_SMALLEST_BLOCK<<'''1''')/uiPelsInPart ) ) == 0)
     28) {
    2429   xEdgeFilterChroma   ( pcCU, uiAbsZorderIdx, uiDepth, edgeDir, iEdge );
    2530}
     31}}}
    2632
    2733Here there is from my understanding a difference in the text and software exist as the text (8.7.2.5.2) edge (2) also is filtered in 422 but not in 420. It might be a misinterpretation from the text.

This list contains all users that will be notified about changes made to this ticket.

These roles will be notified: Reporter, Owner, Subscriber, Participant

  • cchi-jctvc(Reporter, Participant)
  • David Flynn(Subscriber, Participant)
  • jct-vc@…(Subscriber)
  • Karl Sharman(Subscriber, Participant)
  • karl.sharman@…(Always)
  • Karsten Suehring(Participant, Always)