[FFmpeg-devel] [PATCH] fate/libswresample: add a test downmixing with a custom order layout

[James Almer]

On 2/6/2025 11:26 AM, Michael Niedermayer wrote:
> Hi James
> 
> On Wed, Feb 05, 2025 at 11:46:32AM -0300, James Almer wrote:
>> Signed-off-by: James Almer <jamrial at gmail.com>
>> ---
>>   tests/fate/libswresample.mak       | 8 ++++++++
>>   tests/filtergraphs/custom_rematrix | 1 +
>>   2 files changed, 9 insertions(+)
>>   create mode 100644 tests/filtergraphs/custom_rematrix
>>
>> diff --git a/tests/fate/libswresample.mak b/tests/fate/libswresample.mak
>> index a1e5ab91fa..aa4438a4bd 100644
>> --- a/tests/fate/libswresample.mak
>> +++ b/tests/fate/libswresample.mak
>> @@ -1097,5 +1097,13 @@ fate-swr-audioconvert: CMP = stddev
>>   fate-swr-audioconvert: FUZZ = 0
>>   
>>   FATE_SWR += $(FATE_SWR_AUDIOCONVERT-yes)
>> +
>> +FATE_SWR_CUSTOM_REMATRIX-$(call FILTERDEMDECENCMUX, ARESAMPLE CHANNELMAP AFORMAT, WAV, PCM_S16LE, PCM_S16LE, PCM_S16LE) += fate-swr-custom-rematrix
>> +fate-swr-custom-rematrix: tests/data/asynth-44100-8.wav tests/data/filtergraphs/custom_rematrix
>> +fate-swr-custom-rematrix: CMD = md5 -i $(TARGET_PATH)/tests/data/asynth-44100-8.wav -/filter_complex $(TARGET_PATH)/tests/data/filtergraphs/custom_rematrix -map [OUT] -f s16le
>> +fate-swr-custom-rematrix: CMP = oneline
>> +fate-swr-custom-rematrix: REF = 2a14a44deb4ae26e3b474ddbfbc048f8
> 
> not a huge fan of md5 based testing of downmixing as it provides just a
> "same" vs "different"

Afaict, downmixing this sample to stereo using either 
FL+FR+FC+LFE+SL+SR+BL+BR (custom order) and 7.1 (native) as input layout 
will result in the same output given that both side and back channels 
are assigned the exact same coeffs, so this test is mainly just making 
sure swr doesn't reject the non-native input layout.

> 
> but patch ok
> 
> thx
> 
> [...]
> 
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".

-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature.asc
Type: application/pgp-signature
Size: 495 bytes
Desc: OpenPGP digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20250206/0ba1aba9/attachment.sig>