[Feature] #2721
Unanswered
ZhangLearning
asked this question in
General
[Feature]
#2721
Replies: 1 comment 1 reply
-
你好,请问你是觉得这样的处理不高效么? |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
What is the feature?
你好,我发现了关于 box_iou_rotated.py 中不合理的地方,根据接口描述:
Return the ious betweens boxes. If
aligned
isFalse
, the shape of ious is (N, M) else (N,).即返回值在 is (N, M) or (N,)。
因此建议修改 :
mmcv/mmcv/ops/box_iou_rotated.py
Line 136 in 66c5515
为
ious = bboxes1.new_zeros([rows , cols])
并删除 146 和 147 行的 view 操作:
mmcv/mmcv/ops/box_iou_rotated.py
Line 147 in 66c5515
根据我前期的测试,可能还需要修改 CPU kernel 的实现,位置在:
mmcv/mmcv/ops/csrc/pytorch/cpu/box_iou_rotated.cpp
Line 24 in 66c5515
Any other context?
No response
Beta Was this translation helpful? Give feedback.
All reactions