Opened 2 years ago
Closed 2 years ago
#369 closed defect (fixed)
Typos in JVET-O2001
Reported by: | Tomohiro Ikai | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | spec | Version: | |
Keywords: | Cc: | ksuehring, bbross, XiangLi, fbossen, jvet@… |
Description
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.
Change history (3)
comment:1 Changed 2 years ago by yyasugi
comment:2 Changed 2 years ago by bbross
Fixed in D6 vC
comment:3 Changed 2 years ago by bbross
- Resolution set to fixed
- Status changed from new to closed
Fixed in D6 vC
I would like to report some other typos.
7.4.3.1 Decoding parameter set RBSP semantics
In (7-7), missing ']' at the end of line (bold text)
if ( i = = 0)
else if( subpic_grid_idx[ i ][ j ] != subpic_grid_idx[ i – 1 ][ j ] )
{
}
8.5.2.1 equation (8-268)
whould be