Grid-Anchor-based-Image-Cropping icon indicating copy to clipboard operation
Grid-Anchor-based-Image-Cropping copied to clipboard

The author fixed a bug in calculating Acc.

Open HuiZeng opened this issue 4 years ago • 3 comments

We recently found an implementation bug in calcualating the Acc indexes which were slightly different from the definition in our paper. Fortunately, this bug does not affect any of the conclusions in our paper. We have corrected this bug in the Matlab code and updated the corresponding values in the revised paper using blue-color font. We apologize for the confusion to readers. The detailed changes in code can be found here: https://github.com/HuiZeng/Grid-Anchor-based-Image-Cropping/commit/5818679685fa1e11ee143f65cc3eb495622349bc#diff-d2f9e0c4e433d155b38659652521d46a

The original code actually calculates the following Acc: image rather than the real definition in our paper: image

HuiZeng avatar Jul 10 '19 09:07 HuiZeng

Hi,I found that the acc2/5 is lager than the acc3/5 at the line VEN of the table 5 in your updated paper. Is that right? I think it is impossible according to definition of ACC in your paper.

SungYK avatar May 06 '20 14:05 SungYK

Hi,I found that the acc2/5 is lager than the acc3/5 at the line VEN of the table 5 in your updated paper. Is that right? I think it is impossible according to definition of ACC in your paper.

Hi, it is possible. Let me give one example, suppose we have only one test image and the returned 4 crops have ranks {1, 10, 2, 4} in groundtruth, then: acc2/5 is (1+0)/2=0.5, acc3/5 = (1+0+1)/3 = 0.67,
acc4/5=(1+0+1+1)/4=0.75.

HuiZeng avatar May 08 '20 08:05 HuiZeng

Thanks for reply. I get that.

SungYK avatar May 08 '20 08:05 SungYK