[FFmpeg-devel] [PATCH] tests/checkasm/checkasm: Provide verbose failure information on float_near_abs_eps() failures
James Almer
jamrial at gmail.com
Sat Apr 14 16:27:44 EEST 2018
On 4/14/2018 9:21 AM, Michael Niedermayer wrote:
> On Fri, Apr 13, 2018 at 12:19:38AM -0300, James Almer wrote:
>> On 4/12/2018 9:34 PM, Michael Niedermayer wrote:
>>> This will make understanding failures and adjusting EPS easier
>>>
>>> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
>>> ---
>>> tests/checkasm/checkasm.c | 6 +++++-
>>> 1 file changed, 5 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/tests/checkasm/checkasm.c b/tests/checkasm/checkasm.c
>>> index 20ce56932f..8a3e24f100 100644
>>> --- a/tests/checkasm/checkasm.c
>>> +++ b/tests/checkasm/checkasm.c
>>> @@ -294,8 +294,12 @@ int float_near_ulp_array(const float *a, const float *b, unsigned max_ulp,
>>> int float_near_abs_eps(float a, float b, float eps)
>>> {
>>> float abs_diff = fabsf(a - b);
>>> + if (abs_diff < eps)
>>> + return 1;
>>>
>>> - return abs_diff < eps;
>>> + fprintf(stderr, "test failed comparing %f with %f (abs diff=%f with EPS=%f)\n", a, b, abs_diff, eps);
>>
>> Maybe %g instead? I may be better to print small values, but I'm not
>> sure. LGTM in any case.
>
> %g is a good idea, ill change it to that
>
>
>>
>> A few tests also output a custom log message like this one, so it may be
>> a good idea to remove them now that it's done in general.
>
> agree, not sure which messages exactly you mean though
>
> will apply so we can easily gather more information about some failures
The synth filter test prints a custom error message similar to the one
you're adding, and other modules probably do the same. I'll see about
removing them now that this is printed by float_near_abs_eps() itself.
>
> thanks
>
> [...]
>
>
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
More information about the ffmpeg-devel
mailing list