-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Modified triangle and dot in annotators-core #1294
Conversation
I have read the CLA Document and I sign the CLA |
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 starting point - I like the testing colab and the resulting annotation, but there's way to improve the code. I left a few comments 😉
Signed-off-by: Onuralp SEZER <thunderbirdtr@gmail.com>
supervision/annotators/core.py
Outdated
@@ -898,6 +901,7 @@ def annotate( | |||
else custom_color_lookup, | |||
) | |||
center = (int(xy[detection_idx, 0]), int(xy[detection_idx, 1])) | |||
cv2.circle(scene, center, self.radius, (0, 0, 0), self.outline_thickness) |
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.
Let's only do this when self.outline_thickness
is not 0, just like in the triangle annotator
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.
Yes @LinasKo, will add a condition statement here. Updating it and submiting.
Hi @LinasKo, please check the latest colab notebook These are the final changes. Please let me know about further changes if needed. Thanks |
62d0a60
to
272a9cb
Compare
Signed-off-by: Onuralp SEZER <thunderbirdtr@gmail.com>
4040b4a
to
eec3615
Compare
Thank you, @dsaha21 and @onuralpszr! It's great to have such a nice contribution in the repo this quick 😉 |
Description
Adding changes as described in the ISSUE
The modifications upon the existing code base might result for a more clear visualisation
Testing
Also adding the colab file for checking and testing.
Thanks.