Xiang Li authored 5 years ago and Karsten Suehring committed 5 years ago

Fixed the encoder bug which leads to incorrect BVd signaling. The incorrect BVd triggers exception on valid BV checking.

compositeRefEnabled is an encoder only option. It should not be stored in the SPS.
This commit is the last to remove unused parameters from SPSNext. Thus also
all comments related to SPSNext are removed.

AMVR (formely IMV) now has only one mode, which used to be mode #2
- remove SPS imv_mode parameter
- remove encoder configuration for mode #1 (mode #2 replaces mode #1)
- adapt config files to refer tp mode #1
- rename SPS syntax elements and access functions to AMVR

Note: IMV to AMVR renaming in other parts of the software should be done by proponents.

- 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

- removes old getIBCMode functions from SPSNext
- fixes references to this function in
  * reshaper
  * hash based ME

Note: SPSNext is going to be removed. Fixes can also be applied with the removal patch.