Skip to content
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

[Refactor] Gliding vertex (qbox version) #520

Merged
merged 11 commits into from
Oct 14, 2022

Conversation

zytx121
Copy link
Collaborator

@zytx121 zytx121 commented Sep 21, 2022

nms_qbox and box_iou_qbox are required in MMCV

  • Align test
  • Align train
  • UT

Experiments

gliding_vertex_r50_fpn_1x_dota_le90.py
nms_rotated & merge(rbox)
report: 73.23
AP50: 0.7372587373211192
AP75: 0.37680789715768237
mAP: 0.39785740345399534

nms_quadri & merge(rbox) iou_thr=0.1
AP50: 0.7362829357289702
AP75: 0.3749092724927537
mAP: 0.3978762499250482

nms_quadri & merge(qbox)
AP50: 0.7061511441138075
AP75: 0.3754383915917645
mAP: 0.38538783994321235

@zytx121 zytx121 changed the title [Refactor] Gliding vertex (Qboxe) [Refactor] Gliding vertex (qbox version) Sep 21, 2022
@zytx121 zytx121 changed the base branch from main to dev-1.x September 21, 2022 08:58
liuyanyi
liuyanyi previously approved these changes Oct 14, 2022
@RangiLyu RangiLyu merged commit f551291 into open-mmlab:dev-1.x Oct 14, 2022
zytx121 added a commit to Justice-Eternal/mmrotate that referenced this pull request Oct 16, 2022
* Update s2anet_r50_fpn_fp16_1x_dota_le135.py

* add UT

* Update test_delta_midpointoffset_rbbox_coder.py

* update

* fix lint

* update

* update

* add rbox version

* fix

* add UT

* fix lint
@zytx121 zytx121 mentioned this pull request Nov 5, 2022
triple-Mu pushed a commit to triple-Mu/mmrotate that referenced this pull request Jan 31, 2023
* Update s2anet_r50_fpn_fp16_1x_dota_le135.py

* add UT

* Update test_delta_midpointoffset_rbbox_coder.py

* update

* fix lint

* update

* update

* add rbox version

* fix

* add UT

* fix lint
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants