[Feature] Full angle range

calcoloergosum commented 1 month ago

NOTE: This PR alone does not make training work.

Thank you for your amazing codebase.

Motivation

Resolve #386 #565 #902 #1045 . Sometimes we want the full range of angle.

Modification

Added new angle_version called "full360".

Use cases (Optional)

Anything that is not rotation invariant shaped objects. For example:

letters
animal with head/tail. Yudo
arrows
...

Note

Introducing full angle range requires specific convention regarding the order of points in a bounding box. For code in this PR, bounding box is defined to be the clockwise - (lt, rt, br, bl) or [(-,-), (-,+), (+,+), (+, -)].
First time contributing. Please let me know if I am doing something off.
To make it work, RBBoxAssigner should be modified accordingly.

Checklist

[x] Pre-commit or other linting tools are used to fix the potential lint issues.
[x] The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
[ ] The documentation has been modified accordingly, like docstring or example tutorials.

CLAassistant commented 1 month ago

All committers have signed the CLA.

calcoloergosum commented 3 weeks ago

I just realized that this repo already is working on 360 rotation detection :P

yangxue0827 commented 3 weeks ago

I think your implementation is better and more flexible.

zytx121 commented 2 weeks ago

@calcoloergosum Thank you for your contribution. We will incorporate your angle representation into the main branch as one of the default angle representations.

open-mmlab / mmrotate