-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Added cv_detect_contour module #1798
Conversation
extern float contour_d_x, contour_d_y, contour_d_z; | ||
|
||
// values for thresholding the contour | ||
extern int lower_y, upper_y, lower_u, upper_u, lower_v, upper_v; |
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.
these should probably be either passed as args to find_contour or at least grouped in a struct to not pollute the whole namespace with these rather generic names...
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.
Agreed! Done.
IMHO same goes for the whole contour result stuff... |
Done! |
extern float contour_d_x, contour_d_y, contour_d_z; | ||
struct contour_estimation{ | ||
|
||
float contour_d_x, contour_d_y, contour_d_z; |
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 fix indentation in this file...
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.
done
Nice! Thank you very much. |
Revamped #1785
Now it works, code looks nice, no warnings and actually compiles.