-
-
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
Computing haar features #1444
Computing haar features #1444
Conversation
No merges on PRs, please |
Also, please read through this page here: http://scikit-image.org/docs/stable/contribute.html |
Sorry, I was a bit confused when checking out the feature branch. Thanks for the headsup, will sure follow it from the next time. And I have gone through the contribution workflow. |
3e9c8b0
to
190802e
Compare
https://travis-ci.org/scikit-image/scikit-image/builds/56050940 I am facing this problem. It would be great if someone could help me solve this. |
Re-running the tests
@spark1729 I restarted the Travis build now that #1457 is merged - that should sort the flake8 error. It doesn't look like AppVeyor can be restarted. AppVeyor had trouble cloning the repository, likely due to the ongoing DDoS against GitHub, so if we see that error sporadically we can ignore it. |
A rebase will be needed to clear the error. |
A rebase will be needed to clear the error (I think). |
It returns the specific haar feature, evaluated in a 24*24 frame, | ||
as a single array. | ||
|
||
Input |
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.
Please use Numpy documentation standard https://github.com/numpy/numpy/blob/master/doc/HOWTO_DOCUMENT.rst.txt#docstring-standard
with Parameters, Returns, etc. sections
Hi @spark1729 could you please explain what this |
Implement Haar-like features as discussed in Issuee #1431 .