Opened 12 years ago Closed 12 years ago #734 closed defect (fixed)HM8.0 bug in SAO + PCM + lossless
Description
1) In HM8.0, when ALF is removed from the software, TComAdaptiveLoopFilter::PCMLFDisableProcess() was also removed, although it is necessary to prevent SAO from altering PCM or lossless-coded CUs.
2) Even if we revert the function, it has a bug as discribed below.
Attachments (1)Change History (4)comment:1 Changed 12 years ago by DefaultCC Plugin
Changed 12 years ago by kchonocomment:2 Changed 12 years ago by kchonocomment:3 Changed 12 years ago by ksuehring
Note: See
TracTickets for help on using
tickets. | This list contains all users that will be notified about changes made to this ticket. These roles will be notified: Reporter, Owner, Subscriber, Participant
|
The issue is true and the bug fix is correct. Attached zip contains two bug fix patches to HM-8.0: "ticket734-fix1.patch" fixes the filtering condition according to (2), "ticket734-fix2.patch" on top of "ticket734-fix1.patch" copies the disabled functions of TComAdaptiveLoopFilter to TComSampleAdaptiveOffset.
So "ticket734-fix2.patch" can fix all issues without reverting ALF code.