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.
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
Test coverage #351
Test coverage #351
Conversation
@@ -98,7 +98,7 @@ def find_contours(array, level, | |||
""" | |||
array = np.asarray(array, dtype=np.double) | |||
if array.ndim != 2: | |||
raise RuntimeError('Only 2D arrays are supported.') | |||
raise TypeError('Only 2D arrays are supported.') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ValueError?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed
Thanks! |
|
||
## def func(Y): | ||
## return ssim(X, Y, dynamic_range=255) | ||
def test_ssim_grad(): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you add a note that this test is known to fail on OSX 10.6?
See the following thread for more information:
https://groups.google.com/d/topic/scikit-image/hOJhyDRzOG4/discussion
I never tracked down the source of the error, but @stefanv was able to reproduce the failure on a similar system so I don't think it's just me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note added.
+1 |
Hmm, the tests ran fine on my system, but the Travis CI fails: I didn't notice until now that Travis isn't testing PRs anymore. |
I re-enabled it this morning, after the repo move. |
Improvements for test coverage of several functions / packages.
At the same time I fixed some bugs I came across.