You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In the current implementation of v in compute_ciou function (keras/src/layers/preprocessing/image_preprocessing/bounding_boxes/iou.py), the (4 / math.pi**2) term is included inside power and so gets squared together with the arctan term.
However, according to equation (8) in the original CIoU paper, (4 / math.pi**2) should NOT be squared (i.e., it should be outside the power op) - only the arctan term should be squared. This is correctly reflected in Ultralytics' implementation of CIoU here.
The text was updated successfully, but these errors were encountered:
Describe the bug
In the current implementation of
v
incompute_ciou
function (keras/src/layers/preprocessing/image_preprocessing/bounding_boxes/iou.py), the(4 / math.pi**2)
term is included insidepower
and so gets squared together with the arctan term.However, according to equation (8) in the original CIoU paper,
(4 / math.pi**2)
should NOT be squared (i.e., it should be outside thepower
op) - only the arctan term should be squared. This is correctly reflected in Ultralytics' implementation of CIoU here.The text was updated successfully, but these errors were encountered: