-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
hough_line_peaks fix for corner case with optimal angle=0 #6271
Conversation
Hello @grlee77! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:
|
@@ -63,7 +62,7 @@ def hough_line_peaks(hspace, angles, dists, min_distance=9, min_angle=10, | |||
min_ydistance=min_distance, | |||
threshold=threshold, | |||
num_peaks=num_peaks) | |||
if a.any(): | |||
if a.size > 0: |
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.
This is the line fixes the bug. the other modified lines are flake8/isort fixes and the new test case
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.
Good catch!
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.
Thank you @grlee77 😉
@@ -63,7 +62,7 @@ def hough_line_peaks(hspace, angles, dists, min_distance=9, min_angle=10, | |||
min_ydistance=min_distance, | |||
threshold=threshold, | |||
num_peaks=num_peaks) | |||
if a.any(): | |||
if a.size > 0: |
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.
Good catch!
@meeseeksdev backport to v0.19.x |
…ith optimal angle=0
…1-on-v0.19.x Backport PR #6271 on branch v0.19.x (hough_line_peaks fix for corner case with optimal angle=0)
Description
closes #4129, #6187
An array,
a
with a single element equal to zero is possible and is not handled properly inhough_line_peaks
. This PR fixes that and adds a test case based on the example from #6187.Checklist
./doc/examples
(new features only)./benchmarks
, if your changes aren't covered by anexisting benchmark
For reviewers
later.
__init__.py
.doc/release/release_dev.rst
.example, to backport to v0.19.x after merging, add the following in a PR
comment:
@meeseeksdev backport to v0.19.x
run-benchmark
label. To rerun, the labelcan be removed and then added again. The benchmark output can be checked in
the "Actions" tab.