[non-BE] Resolve "Inconsistent default reverb configurations"
- Related issues:
- Requested reviewers: @tamarapu, @pihlajakuja
Reason why this change is needed
There are three different default reverb parameters for different rendering paths:
- Parametric renderer parametricReverberationTimes
- Fast convolution renderer fastReverberationTimes
- CREND / TD Renderer ivas_reverb_default_RT60
This results in different synthesized acoustic environments depending on rendering type without using custom reverb configuration.
Proposed fix
The sparse frequency domain reverb initialization uses default reverb times and energies in case no custom roomAcoustics parameters are set. This results in applying reverberation times and energies from impulse responses by default, which are different for ParamBin and FastConv renderers and obviously different from default room acoustics parameters. The proposed fix applies default room acoustics configuration in case of BINAURAL_ROOM_REVERB
output and associates the use of IR related reverberation times and energies with BINAURAL_ROOM_IR
output only.
Related to #587, #1158 (closed)