Changes between Initial Version and Version 3 of Ticket #1230

Timestamp:

Feb 17, 2014, 6:52:49 AM (10 years ago)

Author:

davidf

Comment:

Ticket #1230 – Description

@@ -3 +3 @@
 there 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:
 
+{{{
       luma 16x16                
 -----------------------    <- considered for filtering (1)
@@ -15 +16 @@
 |          |          |
 -----------------------
+}}}
 
 For the corresponding chroma part for both 420 and 422 only (1) is considered and (2) is not considered:
 
 TComLoopFilter.cpp:223
-
-if ( chFmt!=CHROMA_400 && (bAlwaysDoChroma || (uiPelsInPart>DEBLOCK_SMALLEST_BLOCK) || (iEdge % ( (DEBLOCK_SMALLEST_BLOCK<<'''1''')/uiPelsInPart ) ) == 0 ) )    
-{
+{{{
+if ( chFmt!=CHROMA_400
+ && (bAlwaysDoChroma
+  || (uiPelsInPart>DEBLOCK_SMALLEST_BLOCK)
+  || (iEdge % ( (DEBLOCK_SMALLEST_BLOCK<<'''1''')/uiPelsInPart ) ) == 0)
+) {
    xEdgeFilterChroma   ( pcCU, uiAbsZorderIdx, uiDepth, edgeDir, iEdge );
 }
+}}}
 
 Here 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.