Opened 17 years ago

Closed 17 years ago

#126 closed defect (fixed)

Validate_okushiri unit test fails

Reported by: ole Owned by: ole
Priority: highest Milestone:
Component: Functionality and features Version: 1.0
Severity: major Keywords:
Cc:

Description

Auto validation fails with the following error message (in revision 4168). It worked yesterday 10th of January.

cd ./okushiri_tank_validation; python validate_okushiri.py test_that_output_is_as_expected (main.Test_Okushiri) ... Traceback (most recent call last):

File "run_okushiri.py", line 50, in ?

use_cache=True)

File "/home/ole/inundation/anuga_core/source/anuga/abstract_2d_finite_volumes/domain.py", line 330, in set_quantity

self.quantities[name].set_values(*args, kwargs)

File "/home/ole/inundation/anuga_core/source/anuga/abstract_2d_finite_volumes/quantity.py", line 361, in set_values

use_cache = use_cache)

File "/home/ole/inundation/anuga_core/source/anuga/abstract_2d_finite_volumes/quantity.py", line 747, in set_values_from_file

verbose=verbose)

File "/home/ole/inundation/anuga_core/source/anuga/fit_interpolate/fit.py", line 490, in fit_to_mesh

verbose=verbose)

File "/home/ole/inundation/anuga_core/source/anuga/fit_interpolate/fit.py", line 329, in fit

load_file_now=False)):

File "/home/ole/inundation/anuga_core/source/anuga/geospatial_data/geospatial_data.py", line 720, in next

max_read_lines=self.max_read_lines)

File "/home/ole/inundation/anuga_core/source/anuga/geospatial_data/geospatial_data.py", line 834, in _read_csv_file_blocking

header.pop(0)

IndexError?: pop from empty list ERROR

====================================================================== ERROR: test_that_output_is_as_expected (main.Test_Okushiri)


Traceback (most recent call last):

File "validate_okushiri.py", line 29, in setUp

assert res == 0

AssertionError?


Ran 1 test in 61.172s

FAILED (errors=1)

Change History (5)

comment:1 Changed 17 years ago by duncan <duncan.gray@…>

  • Resolution set to fixed
  • Status changed from new to closed

Ole, could you check if it works now

comment:2 Changed 17 years ago by ole

  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:3 Changed 17 years ago by ole

  • Owner changed from duncan to ole
  • Status changed from reopened to new

comment:4 Changed 17 years ago by ole

  • Status changed from new to assigned

comment:5 Changed 17 years ago by ole

  • Resolution set to fixed
  • Status changed from assigned to closed

Changed the criteria and loosened the tolerances (after thorough inspections of timeseries on three platforms). The test passed on nautilus (Linux32), cyclone (Linux64) and Windows32.

This was completed in changeset:4340

Note: See TracTickets for help on using tickets.