Opened 4 years ago

Closed 4 years ago

#975 closed defect (fixed)

Typos in JVET-Q2001-vE

Reported by: Tomohiro Ikai Owned by:
Priority: minor Milestone:
Component: spec Version:
Keywords: Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…

Description

7.3.10.1 General slice data syntax

entropy_coding_sync_enabled_flag
should be
sps_entropy_coding_sync_enabled_flag

7.4.11.7 Merge data semantics

In mmvd_merge_flag[ x0 ][ y0 ], paramters should be parameters

8.4.5.2.13 Specification of INTRA_LT_CCLM, INTRA_L_CCLM and INTRA_T_CCLM intra prediction mode

pDstY[ x ][ y ] = pY[ x ][ y ] (360)
should be
pDsY[ x ][ y ] = pY[ x ][ y ] (360)

8.5.5.1 General
mvL0[ xSbIdx ][ ySbIdx ] and mvL0[ xSbIdx ][ ySbIdx ]
should be
mvL0[ xSbIdx ][ ySbIdx ] and mvL1[ xSbIdx ][ ySbIdx ]

8.5.5.3 Derivation process for subblock-based temporal merging candidates
the location ( xCtr, yCtr ) of the below-right center sample
should be
the location ( xCtrCb, yCtrCb ) of the below-right center sample

8.5.6.5 Bi-directional optical flow prediction process

gradientHL0[ x ][ y ] = ( predSamplesL0[ hx + 1 ][vy] >> shift1 ) − (993)

( predSampleL0[ hx − 1 ][ vy ] ) >> shift1 )

should be
gradientHL0[ x ][ y ] = ( predSamplesL0[ hx + 1 ][vy] >> shift1 ) − (993)

( predSampleL0[ hx − 1 ][ vy ] >> shift1 )

Note: ")" should be removed. The same issue also exists in (994), (995), (996).

8.7.4 Transformation process for scaled transform coefficients
8.7.4.1 General

lfnst_idx[ x0 ][ y0 ] should be lfnst_idx

8.8.5 Adaptive loop filter process
8.8.5.1 General

the coding tree block filtering process for luma samples as specified in clause 8.8.5.2 is invoked with recPictureL, alfPictureL
should be
the coding tree block filtering process for luma samples as specified in clause 8.8.5.2 is invoked with recPictureL
Note: alfPictureL is not used as an input.

as specified in clause 8.8.5.4 is invoked with recPicture set equal to recPictureCb, alfPicture set equal to alfPictureCb,
should be
as specified in clause 8.8.5.4 is invoked with recPicture set equal to recPictureCb

as specified in clause 8.8.5.4 is invoked with recPicture set equal to recPictureCr, alfPicture set equal to alfPictureCr,
should be
as specified in clause 8.8.5.4 is invoked with recPicture set equal to recPictureCr

Note: alfPictureCb,Cr is not used as an input.

8.8.5.2 Coding tree block filtering process for luma samples

– a filtered reconstructed luma picture sample array alfPictureL,
should be removed

Note: alfPictureL is only used as an outout, not an input.

8.8.5.4 Coding tree block filtering process for chroma samples
– a filtered reconstructed chroma picture sample array alfPicture,
should be removed

Note: alfPicture is only used as an outout, not an input.

Change history (2)

comment:1 Changed 4 years ago by yk

Only on the first aspect: Good catch! Thanks! Will be fixed in JVET-R2001-v8.

Other aspects are left for Ben/Jianle/Shan.

comment:2 Changed 4 years ago by bbross

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

Good catches, thanks!

The remaining ones will be fixed in the first version of D10 except the one related to alfPicture.
Here the alf picture are initialized before the CTU loop and passed to process for each CTU so that just the porition of the filtered CTU is overwritten and outputted. Therfore the input is needed because it is processed on a per CTU basis.

Note: See TracTickets for help on using tickets.