It cleans up old ISP code that is no longer used anymore. The code was used to allow only one ISP split depending on the block size. However, this functionality is not needed anymore after the adoption of JVET-N0308 during the Geneva Meeting in March, which limited the usage of ISP to CUs with a maximum size of 64x64 samples. This clean-up eliminates various unnecessary checks and improves the readability of the code.

JVET-O0219: LFNST transform set selection for a CCLM coded block. Remove duplicate code. Fix a bug due to convert MIP mode to convensional mode

JVET-O0219: LFNST transform set selection for a CCLM coded block. Remove duplicate code. Fix a bug due to convert MIP mode to convensional mode

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

- getTrTypes function is moved to TrQuant class

- Implicit transform derivation
- DST-7 is applied horizontally/vertically as long as the number of (luma) samples are less than or equal to 16 in a row/column, otherwise DCT-2 is applied.

keep max TU in #define for possibility of keeping the parameter

TU max/min size are constants in the VVC draft. Reflecting this in the software.

This is for the case where JVET_M0464_UNI_MTS is disabled

- all elements of SPSNext are moved into SPS
- syntax elements of SPS are sorted according to draft (as far as possible)
- comments were added where software and draft diverge

calculate and check shifts only when they are actually used

There is a conflict with JVET_M0427_INLOOP_RESHAPER that is triggering a debug assertion that checks that the width is not 1.