[svn-r20384] Purpose:
Fixed Bug 2184 - GMQS: h5diff - incorrect calculation code for
--use-system-epsilon option
Description:
Merged from HDF5 1.8 branch r20369.
Fixed h5diff for --use-system-epsilon option: the calculation changed
from ( |a - b| / b ) to ( |a - b| ). This was decided for better
performance and was corrected only in HDF5 trunk, so 1.8 got updated.
Also comments for equal_XXX() function were updated correctly.
Also help page and RM got updated correctly.
Also add test cases for testing the differences w/wo the option.
Tested:
jam (linux32-LE), amani (linux64-LE), heiwa (linuxppc64-BE), tejeda (mac32-LE), linew (solaris-BE), Windows (32-LE)
This commit is contained in:
@@ -602,8 +602,8 @@ rem ############################################################################
|
||||
|
||||
rem 11. floating point comparison
|
||||
rem Not tested on Windows due to difference in formatting of scientific
|
||||
rem notation --SJW 8/23/07
|
||||
call :testing h5diff_101.txt -v %srcfile1% %srcfile1% g1/d1 g1/d2
|
||||
rem notation (101, 102) --SJW 8/23/07
|
||||
call :testing %h5diff% -v %srcfile1% %srcfile1% g1/d1 g1/d2
|
||||
rem call :tooltest h5diff_101.txt -v %file1% %file1% g1/d1 g1/d2
|
||||
call :results -SKIP-
|
||||
|
||||
@@ -611,6 +611,16 @@ rem ############################################################################
|
||||
rem call :tooltest h5diff_102.txt -v %file1% %file1% g1/fp1 g1/fp2
|
||||
call :results -SKIP-
|
||||
|
||||
rem Not tested on Windows due to difference in formatting of scientific
|
||||
rem notation with other OS. printf("%g") (103, 104)
|
||||
call :testing %h5diff% -v --use-system-epsilon %srcfile1% %srcfile1% g1/d1 g1/d2
|
||||
rem call :tooltest h5diff_103.txt -v --use-system-epsilon %file1% %file1% g1/d1 g1/d2
|
||||
call :results -SKIP-
|
||||
|
||||
call :testing %h5diff% -v --use-system-epsilon %srcfile1% %srcfile1% g1/fp1 g1/fp2
|
||||
rem call :tooltest h5diff_102.txt -v --use-system-epsilon %file1% %file1% g1/fp1 g1/fp2
|
||||
call :results -SKIP-
|
||||
|
||||
rem New option added #1368(E1) - ADB 2/5/2009
|
||||
rem not compable -c flag
|
||||
call :testing %h5diff% %srcfile2% %srcfile2% g2/dset1 g2/dset2
|
||||
|
||||
Reference in New Issue
Block a user