[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
-
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