Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

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

Closed
lukecampbell opened this issue Jun 12, 2014 · 2 comments
Closed

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

lukecampbell opened this issue Jun 12, 2014 · 2 comments

Comments

@lukecampbell
Copy link
Member 

======================================================================
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
Copy link
Member Author

caseybryant#14

@lukecampbell
Copy link
Member Author

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?

caseybryant added a commit to caseybryant/coverage-model that referenced this issue Jun 17, 2014
@caseybryant
M117 - Bug fixes
3af4a8d 
Issue ooici#174 - Reimplemented ArrayType to be an ndarray of homogenous type and fixed inner length.
Issue ooici#181 - Should be resolved with ooici#174 changes
Issue ooici#185 - Added support for time_segment=(None,None)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@lukecampbell