Skip to content
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.

Sign up for GitHub

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

Jump to bottom

cv2.minAreaRect(box) #48

Open
eccstartup opened this issue Jun 14, 2018 · 1 comment
Open

cv2.minAreaRect(box) #48

eccstartup opened this issue Jun 14, 2018 · 1 comment

Comments

@eccstartup
Copy link

R2CNN_FPN_Tensorflow/libs/box_utils/coordinate_convert.py

Line 43 in 37da5d7

rect1 = cv2.minAreaRect(box)

See here,

Developer should keep in mind that the returned rotatedRect can contain negative indices when data is close to the containing Mat element boundary.

the use of minAreaRect could be unsafe. And the drawing will crush the program.
@tonyma666
Copy link

I think you can make a bounds check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@eccstartup @tonyma666