[Libreoffice-commits] core.git: adjust rounding decimals to expected values

Stephan Bergmann sbergman at redhat.com
Mon Dec 19 10:03:25 UTC 2016


On 12/16/2016 07:40 PM, Eike Rathke wrote:
>  sc/qa/unit/data/functions/statistical/fods/forecast.ets.add.fods |  655 +++++-----
>  1 file changed, 346 insertions(+), 309 deletions(-)
>
> New commits:
> commit 8bd9e0503ebb6771c543e2fd9a0dc0d2d715f304
> Author: Eike Rathke <erack at redhat.com>
> Date:   Fri Dec 16 19:22:52 2016 +0100
>
>     adjust rounding decimals to expected values
>
>     This may have been the cause for some tinderboxes choking on this test
>     on Windows and Mac if the forecast results slightly differed due to the
>     initialization with random values and the calculation taking place with
>     a different internal floating point precision. So let's try.
>
>     Blindly throwing ROUND(...,12) at all values is wrong anyway because
>     that doesn't take the magnitude of the expected value into account. What
>     we actually want is an accuracy of 12 digits, not an accuracy of 12
>     decimals.
>
>     0.123456789012
>     1.23456789012
>     12.3456789012
>     123.456789012
>     ...
>
>     Change-Id: If88d75277c3d17cfd72f36d0f88d093b63da8add

...apparently doesn't fix the problem; 
<http://ci.libreoffice.org/job/lo_tb_master_mac_dbg/11172/console> still 
failed with

> Testing file:///Users/tdf/lode/jenkins/workspace/lo_tb_master_mac_dbg/sc/qa/unit/data/functions/statistical/fods/forecast.ets.add.fods:
> /Users/tdf/lode/jenkins/workspace/lo_tb_master_mac_dbg/sc/qa/unit/functions_test.cxx:43:StatisticalFunctionsTest::testStatisticalFormulasFODS
> double equality assertion failed
> - Expected: 1
> - Actual  : 0
> - Delta   : 1e-14


More information about the LibreOffice mailing list