id summary reporter owner description type status priority milestone component version resolution keywords cc 369 Typos in JVET-O2001 Tomohiro Ikai "In the latest JVET-O2001-vB, the followings would be typos. == 1. In 7.3.4.1 General HRD parameters syntax HDR would be HRD. {{{ 7.3.4.1 General HDR parameters syntax would be 7.3.4.1 General HRD parameters syntax }}} == 2. In 7.3.8.2 and others MODE_ALL would be MODE_TYPE_ALL == 3. In 7.4.9.7 Merge data semantics ""merge_subblock_flag[ x0 ][ y0 ] is equal to 0."" is duplicated, please remove one of them. {{{ When ciip_flag[ x0 ][ y0 ] is not present, it is inferred as follows: – If all the following conditions are true, ciip_flag[ x0 ][ y0 ] is inferred to be equal to 1: – sps_ciip_enabled_flag is equal to 1. – general_merge_flag[ x0 ][ y0 ] is equal to 1. – merge_subblock_flag[ x0 ][ y0 ] is equal to 0. – regular_merge_flag[ x0 ][ y0 ] is equal to 0. – merge_subblock_flag[ x0 ][ y0 ] is equal to 0. **here** }}} == 4. In 8.5.2.7 Derivation process for merge motion vector difference mMvdL10 would be mMvdL1 {{{ mMvdL10[ 0 ] = Sign( currPocDiffL0 ) = = Sign( currPocDiffL1 ) ? mMvdL1[ 0 ] : −mMvdL1[ 0 ] (8 372) would be mMvdL1[ 0 ] = Sign( currPocDiffL0 ) = = Sign( currPocDiffL1 ) ? mMvdL1[ 0 ] : −mMvdL1[ 0 ] (8 372) }}} == 5. In 8.6.3 Decoding process for IBC blocks xVb would be yVb and % would be & {{{ xVb = ( y + bv[ 1 ] >> 4 ) ) % ( CtbSizeY − 1 ) (8 874) would be yVb = ( y + bv[ 1 ] >> 4 ) ) & ( CtbSizeY − 1 ) (8 874) }}} == 6. In 8.7.4.2 Low frequency non-separable transformation process There're duplicated ""of"" after ""a list"". {{{ – a list of of scaled non-zero transform coefficients x[ j ] with j = 0..nonZeroSize − 1, }}} == 7. In 6.5.2 Up-right diagonal and raster scan order array initialization process Here I'm not so sure, but rasterScan's parameter (x, y) is swapped, isn't it? {{{ rasterScan[ x * blkWidth + y] = i could be rasterScan[ y * blkWidth + x] = i }}} Plus, the rasterScan's naming can be confusing since this is not a usual rasterScan but rasterScan2DiagonalScanPos or something. The variable is used as RasterScanOrder[] for scaling list. " defect closed minor spec fixed ksuehring bbross XiangLi fbossen jvet@…