search

ooici / coverage-model

BSD 2-Clause "Simplified" License
2 stars 9 forks source link

ComplexCoverage get_parameter_values raises if using time segment of (None,None) #185

Closed lukecampbell closed 10 years ago

lukecampbell commented 10 years ago 
======================================================================
ERROR: test_attributes ( coverage_model.test.test_complex_coverage:TestNewComplexCoverageInt.test_attributes )
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/luke/Documents/Dev/code/coi2/extern/coverage-model/coverage_model/test/test_complex_coverage.py", line 901, in test_attributes
    data = ccov.get_parameter_values(time_segment=(None,None), fill_empty_params=True, as_record_array=False).get_data()
  File "/Users/luke/Documents/Dev/code/coi2/extern/coverage-model/coverage_model/coverages/complex_coverage.py", line 71, in get_parameter_values
    current_time_segment = get_overlap(extents, time_segment)
  File "/Users/luke/Documents/Dev/code/coi2/extern/coverage-model/coverage_model/util/extent_utils.py", line 24, in get_overlap
    raise RuntimeError('No overlap')
RuntimeError: No overlap

----------------------------------------------------------------------
lukecampbell commented 10 years ago

https://github.com/caseybryant/coverage-model/pull/14

lukecampbell commented 10 years ago

I'm gonna ignore retrieve requests time segments of None,None for now but can you look and see if this is really an issue?