search

GeoscienceAustralia / eqrm

Automatically exported from code.google.com/p/eqrm
Other
5 stars 4 forks source link

long_check_scenarios.py failing #28

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. python long_check_scenarios.py
2.
3.

What do you see instead?

======================================================================
File difference: java02/java_bedrock_SA_rp[2500pt0].txt
----------------------------------------------------------------------
First line difference;
Standard line: 0.2234152708 0.4619791838 0.1342203986
 Current line: 0.2231965515 0.4832550905 0.1442228567

0 files missing.
Checked 16 files. 12 failed.

This could be due to the big 878 check in that changed the random order but did 
not take the long scenarios into account.

Original issue reported on code.google.com by duncan.g...@gmail.com on 7 Mar 2012 at 11:52

GoogleCodeExporter commented 9 years ago 
when to revision 877. It still fails;

======================================================================
File difference: java02/java_bedrock_SA_rp[2500].txt
----------------------------------------------------------------------
First line difference;
Standard line: 0.2234152708 0.4619791838 0.1342203986
 Current line: 0.2232257411 0.4832935904 0.1441063838

1 files missing.
Checked 15 files. 11 failed.

Revision 850 could be the problem.

Original comment by duncan.g...@gmail.com on 8 Mar 2012 at 12:23

GoogleCodeExporter commented 9 years ago 
849 Works fine.
Check that 850 to 877 gives the same results.
Then 878 to current.

If that is the case, then long_check changed for understandable reasons, and is 
ok.

Original comment by duncan.g...@gmail.com on 8 Mar 2012 at 1:00

GoogleCodeExporter commented 9 years ago 
850 - 878 gives the same results.
Check the most recent.

Original comment by duncan.g...@gmail.com on 8 Mar 2012 at 6:18

GoogleCodeExporter commented 9 years ago 
recent fails;

======================================================================
File difference: java02/java_soil_SA_rp[1000pt0].txt
----------------------------------------------------------------------
First line difference;
Standard line: 0.2188832274 0.5235217895 0.179867318
 Current line: 0.2186527721 0.5233791274 0.1797637315

======================================================================
File difference: java02/java_soil_SA_rp[2500pt0].txt
----------------------------------------------------------------------
First line difference;
Standard line: 0.2655715879 0.6170156789 0.2537869824
 Current line: 0.2659117203 0.6170139632 0.2537347606

======================================================================
File difference: java02/java_bedrock_SA_rp[500pt0].txt
----------------------------------------------------------------------
First line difference;
Standard line: 0.1330086324 0.2859758119 0.07870279025
 Current line: 0.1330080788 0.285907836 0.07883761827

======================================================================
File difference: java02/java_soil_SA_rp[500pt0].txt
----------------------------------------------------------------------
First line difference;
Standard line: 0.1658441603 0.3783611921 0.1349200463
 Current line: 0.165614138 0.380261155 0.1344917299

======================================================================
File difference: java02/java_bedrock_SA_rp[2500pt0].txt
----------------------------------------------------------------------
First line difference;
Standard line: 0.2232257411 0.4832935904 0.1441063838
 Current line: 0.2231965515 0.4832550905 0.1442228567

======================================================================
File difference: java02/java_bedrock_SA_rp[1000pt0].txt
----------------------------------------------------------------------
First line difference;
Standard line: 0.1746743445 0.3876903 0.1046592232
 Current line: 0.1746707757 0.3875882177 0.1046575795

Original comment by duncan.g...@gmail.com on 12 Mar 2012 at 10:52

GoogleCodeExporter commented 9 years ago 
Should test to 893.  894 is a revision where results may change.

Original comment by duncan.g...@gmail.com on 13 Mar 2012 at 12:25

GoogleCodeExporter commented 9 years ago 
so;
849 Works fine.
850 getting Lat Long random values instead of x y random values. Understandable 
change. 
850 - 892 no change 
893 introduced correct code that did not break the standard unit tests, but 
broke the long test.  This is described in issue 2.  apply_threshold_distance 
was using an incorrect index.  This was fixed in revision 893.
893 - head no change (except input file format)

Therefore the current can be set to the standard for the long tests.

Original comment by duncan.g...@gmail.com on 14 Mar 2012 at 12:21

GoogleCodeExporter commented 9 years ago 

Tests updated in revision 996 so this test passes.

Original comment by duncan.g...@gmail.com on 14 Mar 2012 at 4:14