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-N0805: APS for LMCS (using different APS types)

See merge request !577

- The indexing of cost array was corrected by following shuffling pattern of full-RD candidate list employed by MIP.
- ~0.1% coding gain inc. along with ~4% encoding time dec. in AI

This reverts merge request !532

- The indexing of cost array was corrected by following shuffling pattern of full-RD candidate list employed by MIP.
- ~0.1% coding gain inc. along with ~4% encoding time dec. in AI

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

Changed condition for "fast" RA in reduceHadCandList() and renamed variables using camelCase notation

Cosmetic change: Fixed a typo in the name of a variable: "ispIsCurrentWinnder" was replaced with "ispIsCurrentWinner".