-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Connect Oriented Bounding Box to Metrics #1562
Comments
Hi @patel-zeel 👋 I'm moving the IoU integration into metrics here. I believe we spoke about it briefly. Have you done substantial work on it already? If not, I will open it to the community. |
Contribution guidelinesIf you would like to make a contribution, please check that no one else is assigned already. Then leave a comment such as "Hi, I would like to work on this issue". We're happy to answer any questions about the task even if you choose not to contribute. Please share a Google Colab with minimal code to test the new feature. We know it's additional work, but it will speed up the review process. You may use the Starter Template. The reviewer must test each change. Setting up a local environment to do this is time-consuming. Please ensure that Google Colab can be accessed without any issues (make it public). Thank you! 🙏 |
Hi ,I would like to work on this issue. Please @LinasKo assign me this issue |
Hi @LinasKo, feel free to open it for the community. I'd be happy to help other contributors if they face any issues while implementing :) |
@Prasadayus hello task is yours, good luck have fun :) |
Hi @LinasKo I am new to open source contribution. Please assign me this issue as I want to excel in this field |
Hi @abhinav7289A, This issue is already taken. I'll add many new ones next week - keep an eye on our issue board! 😉 |
If an issue is already taken and I propose a better solution, then will you consider it or not? @LinasKo |
Hi @PrakharJain1509 👋 |
Hi @Prasadayus 👋 How's it going? Did you find any time to look at oriented boxes and metrics? 😉 |
@LinasKo Sorry for the late reply, I was very busy this week. I cloned the project, reviewed the oriented box and will make the changes by the end of this week |
Thank you very much! |
@LinasKo I have opened a PR for this issue. Please review it and let me know if any changes are required. |
Thank you very much @PrakharJain1509 ! I'll have a look and review it tomorrow |
I've just realized that @Prasadayus is the original assigned contributor. Your PR will have priority over that of @PrakharJain1509. @Prasadayus, how goes it? Do you think you'll have time to complete this PR? 😉 |
Adding @PrakharJain1509 as an assignee. |
Accepting #1593 as the solution for adding OBB support for the This ticket also involves the |
@LinasKo Sir, Can you review my changes? |
@rahulsamant37, I reviewed your changes in the PR and decided to close it in favor of #1593, as it aligns better with our current direction. Thanks for your effort and looking forward to your future contributions! |
Completed via #1593. Thank you for your contributions, everyone! I hope you found the experience useful and fun! |
Connect Oriented Bounding Box to Metrics
Tip
Hacktoberfest is calling! Whether it's your first PR or your 50th, you’re helping shape the future of open source. Help us build the most reliable and user-friendly computer vision library out there! 🌱
Several new features were recently added to supervision:
Intersection Over Union (IoU) is the starting point when computing these metrics. It determines which detections are considered true positives. However, take a look! The Oriented Box IoU is not supported yet! Help us add support by using
oriented_box_iou_batch
.Helpful links:
The text was updated successfully, but these errors were encountered: