-
Notifications
You must be signed in to change notification settings - Fork 7.1k
[proto] Added functional rotate_bounding_box op #5638
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
Conversation
💊 CI failures summary and remediationsAs of commit d355c7b (more details on the Dr. CI page): ✅ None of the CI failures appear to be your fault 💚
🚧 3 ongoing upstream failures:These were probably caused by upstream breakages that are not fixed yet.
This comment was automatically generated by Dr. CI (expand for details).Please report bugs/suggestions to the (internal) Dr. CI Users group. |
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.
Thanks! LGTM with a couple of non-blocking nits for your consideration.
Co-authored-by: Vasilis Vryniotis <[email protected]>
Summary: * [proto] Added functional rotate_bounding_box op * Fix mypy * Apply suggestions from code review (Note: this ignores all push blocking failures!) Reviewed By: datumbox Differential Revision: D35216794 fbshipit-source-id: ce3f1421d9b0671d2b00d1fb2d1f86e0a0eddabb Co-authored-by: Vasilis Vryniotis <[email protected]> Co-authored-by: Vasilis Vryniotis <[email protected]>
Related to #5514
Description:
rotate_bounding_box
opexpand=True
andcenter is not None
and disabled providedcenter
Results on synthetic images/bboxes:
Code
Compare to D2:
TL;DR: matches the behaviour if expand=False and ~1 pixel difference otherwise.
This is due to cast to int : facebookresearch/detectron2@63ea900#diff-98378cbc4207da5ac829ceff03d62c48abbfc564911ac7d64c2f1d137b4f3474R130 and the way how affine matrix is computed : facebookresearch/detectron2@63ea900#diff-98378cbc4207da5ac829ceff03d62c48abbfc564911ac7d64c2f1d137b4f3474R171
Code