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
QH6228 Bug fix - remove offset from coords #873
QH6228 Bug fix - remove offset from coords #873
Conversation
As I recall, this approach also fixed the bug I found ~6 months ago (#672). @jmetz, can you add your failing test case to the tests in @stefanv, the decreased coverage appears to indicate that the convex hull tests are skipped on Travis. Is there a reason for this? |
@jni - just having a minor compile issue while trying to run my additional testing; am I missing something simple..? gives
|
@jni - OK I worked around that by manually calling Also I can confirm that my workaround also fixed the other previously failing test ( |
@@ -54,6 +54,10 @@ def convex_hull_image(image): | |||
raise ImportError('Could not import scipy.spatial, only available in ' | |||
'scipy >= 0.9.') | |||
|
|||
# Subtract offset |
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.
Trailing whitespace
I noticed a few very minor style issues about unnecessary whitespace; I think I noted them all. Many editors can be set to automatically trim all trailing whitespace on saves, to avoid this in the future. Travis appears to be compiling and running the new file correctly, so I think this is ready to go once that whitespace gets trimmed! |
Thanks for the catch @JDWarner - I modified my vimrc to remove trailing whitespaces on save and updated the corresponding edits accordingly. |
151, 152, 153, 154, 147, 148, 149, 150, 151, 152, 153, 148, 149, | ||
150, 151, 152, 149, 150, 151, 150])) | ||
image = np.zeros((1392, 1040), dtype=bool) | ||
image[ nonzeros ] = True |
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.
PEP8: image[nonzeros]
@jmetz one final style comment, then it's in as far as I'm concerned! |
QH6228 Bug fix - remove offset from coords Fixes issue #871.
@stefanv, I made a call to ignore the Travis error since it was obviously ancillary to the actual build. The previous build was fine, and this last commit only changed spacing between parentheses. Let me know if you'd rather I restart builds even in such cases. |
@jni Master looks happy, so I'm happy. |
Fixes: #871