From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id C4F0843162 for ; Fri, 22 Jul 2022 14:52:08 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 5B59D68B6F5; Fri, 22 Jul 2022 17:52:05 +0300 (EEST) Received: from mail-oi1-f178.google.com (mail-oi1-f178.google.com [209.85.167.178]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 17F7D68B1B5 for ; Fri, 22 Jul 2022 17:51:59 +0300 (EEST) Received: by mail-oi1-f178.google.com with SMTP id w204so5798403oie.7 for ; Fri, 22 Jul 2022 07:51:59 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=message-id:date:mime-version:user-agent:subject:content-language:to :references:from:in-reply-to:content-transfer-encoding; bh=3WbpLHJ5oroDaf9SoIbBAvzmlAJxV0XiOr4EII80Vuc=; b=gArrBLpq2KXXdevd7byhRYWAavboj0zFscUqtwRc1kN5fJlz56/NAPU1DPcTtfHZuc IsFlyf5hnDrp+LY05tRDoSjd/ytGpRt2wxvVN9oaavGdrxAuJ1iESgoOpsTvrcfvwQ/J RGhsk8TlPSGOCcayWBxQYWm7Ul0+hxC0/fer7JU3bgLscs/fh/SroWDkmBIQ9vLMtyDq 8dkNkkmQRPm5ghdnY4YP4ZW7LMtSCLgKoIYhA9luYwMk1i1H6tGmT4ewOfuCHOvY+Xuz eAvxeS9ej02HikY0hqQuo8PI62a95asbtBxELE8ZSkWd/uF1aBH+azdx0WirUZhnA3N1 YNvA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:references:from:in-reply-to :content-transfer-encoding; bh=3WbpLHJ5oroDaf9SoIbBAvzmlAJxV0XiOr4EII80Vuc=; b=hpeopVCvD/vx9fmnppQWK5gI22ppJavj5AK27FtlYr/Vae1Ra0FLLn+gMAGS6I0mSg DDItPbOS2dX7udriDzhBabH5y5DjJHNid60OjKIyssX23+Qq0w544rC4S5CbTy5h0liU nPKIiVbknDh+kZRrn78+kOzfFTUAXeYWbHHBtmKaSs8DLMPJV2QaZ2KtTez5B54J10vA 2YkyytxSV9ZT9TIt6PZb/lFY8vX1wggPkIkHz8gmC1bbABG/kff9PCOcxtQBAZ/MTUuA F0C6dlK4dQ+0v8b8bQ5lX5tiDxVQw3rXHo1E8Y9p7SXa69cvQNyD4DQ+bJG9HiGvwMu7 WUcg== X-Gm-Message-State: AJIora8tCFYgILq0u1jJ1aVw+HcVvOySW5QPh8N/J5vK4QE+vQr94WNZ o2yoah5hYdWdiLGbDMLIWlhYB7PKK0+gOQ== X-Google-Smtp-Source: AGRyM1uXakCVrI+tUqltS5btQVgoKRGYhrXwCzcK5gr+LpvjF7k6RX4REAXpIr5uqSoWDhe68zGN5Q== X-Received: by 2002:aca:b883:0:b0:33a:a346:5343 with SMTP id i125-20020acab883000000b0033aa3465343mr6081013oif.296.1658501516678; Fri, 22 Jul 2022 07:51:56 -0700 (PDT) Received: from [192.168.0.11] ([186.136.131.204]) by smtp.gmail.com with ESMTPSA id i11-20020a056830010b00b0061cb445a5fesm2005492otp.55.2022.07.22.07.51.55 for (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 22 Jul 2022 07:51:56 -0700 (PDT) Message-ID: <3dfc3a5b-6b08-e36c-44e7-40cdf062cb3c@gmail.com> Date: Fri, 22 Jul 2022 11:51:56 -0300 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.11.0 Content-Language: en-US To: ffmpeg-devel@ffmpeg.org References: <20220713175948.1955-1-jamrial@gmail.com> <8b484a4e-ba96-d54e-a4b5-84e1ee5e53f9@gmail.com> From: James Almer In-Reply-To: Subject: Re: [FFmpeg-devel] [PATCH] avcodec/aacdec: don't force HE-AACv2 profile if no PS info is present X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On 7/22/2022 11:23 AM, Andreas Rheinhardt wrote: > James Almer: >> On 7/18/2022 10:57 AM, Andreas Rheinhardt wrote: >>> James Almer: >>>> On 7/14/2022 9:10 AM, Andreas Rheinhardt wrote: >>>>> James Almer: >>>>>> Should fix ticket #3361 >>>>>> >>>>>> Signed-off-by: James Almer >>>>>> --- >>>>>> This also needs an update to some fate ref samples i'll upload before >>>>>> pushing >>>>>> (fate-aac-al_sbr_ps_04_ur and fate-aac-al_sbr_ps_06_ur which are now >>>>>> decoded >>>>>> properly as he_aac mono, so the .s16 files need to be replaced). >>>>>> >>>>> >>>>> We have both a fixed-point AAC as well as a floating point AAC decoder. >>>>> Is there actually a test that tests that the output they produce is >>>>> reasonably close? If not, could we make the test so that the same file >>>>> is decoded once with the fixed-point and once with the floating-point >>>>> decoder and then compared? >>>> >>>> That wouldn't help much, i think. Almost all changes to *_template.c >>>> files are going to affect both decoders, so a breakage would not be >>>> detected if you compare their output with each other as they would both >>>> exhibit it. >>>> >>> >>> I actually thought that the aac_fixed tests used checksums instead of >>> ref files; then changes and breakages would be visible by changes to >>> these files. Apparently I was wrong about that and the ref files are >>> used for both aac and aac_fixed. But a test like the one outlined above >>> would nevertheless obviate the need for a new ref file. >> >> Judging by >> https://git.videolan.org/?p=ffmpeg.git;a=blob;f=tests/fate/aac.mak;h=1743428f544fad8946dba11dd4ecec0630eb70a6;hb=HEAD#l117 >> it seems at least for these samples the fixed decoder does not generate >> a decoded stream comparable to the float one, so I'll just upload a new >> raw pcm file. > > When I decode both of these streams with git master, the left channel is > pretty much identical, yet the right channel of the fixed-point decoder > is silent and the right channel of the floating point decoder is not. > With this patch applied, the result are two mono streams that are pretty > much identical: The test sample created by the floating-point decoder > works with the fixed-point decoder test (if one uncomments and modifies > the latter). So the issue with aac-al_sbr_ps_06_ur is not a reason to > upload new samples. Ok, can you suggest how to add a test that decodes with the fixed point decoder then compares that with the output of the float decoder? Is there a helper in fate.sh already for this? > > - Andreas > > PS: libfdk-aac produces a file that looks pretty much like the floating > point decoder from git master. Are you sure your patch is correct? Yes, they duplicate the single channel in the stream and output it as stereo, something that should be done by a filter if that's what the user wants. Decoding a mono sample should generate a mono stream. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".