As reported in ticket #291, when sig_flag is inferred, the variables
m_tmplCpDiag and m_tmplCpSum1 are not set correctly, but used for
deriving the contexts for par_glag, gt1_flag, and gt3_flag.

The provided modification fixes this issue by calling cctx.sigCtxIdAbs()
also when sig_flag is inferred (except for the last scanning position,
where a different mechanism is used).

There is no impact on coding efficiency.

implements:
- splitting tiles into bricks
- raster-scan tiles
- uniform and non-uniform splits
- all tiles/bricks in one slice
- each brick one slice
- PPS syntax

missing:
- rectangular tiles
- slice header syntax

further todo:
- rework slice mode 2 to use multiple tiles for byte size matching
- lots of renaming (tile->brick)

Nan Hu authored 5 years ago and Karsten Suehring committed 5 years ago

move m_fixedFilterNum to CommonDef.h, rename bRdo and bRedo to isRdo and isRedo, remove redundant stat derivation.

- Removed lfnst_mode context modelling dependency on intra-prediction mode.
- Second bin of lfnst_mode is bypass-coded rather than context-coded.

Moonmo Koo authored 5 years ago and Frank Bossen committed 5 years ago

- Previous name: Reduced Secondary Transform (RST)
- The parameter, LFNST, should be added in cfg file for enabing LFNST, i.e. LFNST = 1

Xiang Li authored 5 years ago and Frank Bossen committed 5 years ago

Merge branch 'jvet-n0400-trianglenum' of https://vcgit.hhi.fraunhofer.de/XiangLi/VVCSoftware_VTM into jvet-n0400-trianglenum

Merge branch 'N0280-ResidualCodingTransformSkip' of https://vcgit.hhi.fraunhofer.de/nguyen/VVCSoftware_VTM into N0280-ResidualCodingTransformSkip

#define JVET_N0127_MMVD_SPS_FLAG                          1