-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
feat(annotator): 🚀 introduce circle annotator #386
feat(annotator): 🚀 introduce circle annotator #386
Conversation
Signed-off-by: Onuralp SEZER <thunderbirdtr@gmail.com>
Hi, @onuralpszr 👋🏻! Would you be so kind as to update this colab or duplicate it and show case how annotator works? It would make my life a lot simpler! 🙏🏻 |
Of course |
Thank you! 🙏🏻 Let me know once you are done ;) |
@onuralpszr, you know you don't need to work on your fork. You can create branches directly in supervision. |
Habits :=) I will use here more okey 👍 :) |
... scene=image.copy(), | ||
... detections=detections | ||
... ) | ||
``` |
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.
Could you add this link to an example annotated frame in docs? https://media.roboflow.com/supervision-annotator-examples/circle-annotator-example.png
In a similar way, we have done it for other annotators.
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.
@SkalskiP I was going to ask that, awesome thank you !!
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.
No worries ;) let me know once you update it. ;)
Cool. One comment, and we can merge. |
Signed-off-by: Onuralp SEZER <thunderbirdtr@gmail.com>
@SkalskiP all done you can merge |
@onuralpszr One more thing. I see that you marked it as Hacktoberfest task. I'm all in favor. Small question: do we need to do anything else? Do you need issue associated with the PR? Is there anything else we need to do to make that PR count to Hacktoberfest? |
"hacktoberfest-accepted" is enough to make it count. |
Awesome! In that case, merging! 🚀 |
Description
We have box annotator and for support different shape I added circle annotator
List any dependencies that are required for this change.
Type of change
Please delete options that are not relevant.
How has this change been tested, please provide a testcase or example of how you tested the change?