simple-faster-rcnn-pytorch icon indicating copy to clipboard operation
simple-faster-rcnn-pytorch copied to clipboard

Two questions about "bbox_tools.py"

Open Williamongh opened this issue 6 years ago • 3 comments
trafficstars

Thanks for your sharing! It helps me a lot. And I have 2 question about "./model/utils/bbox_tools.py".

  1. In "./model/utils/bbox_tools.py":
import numpy as np
import numpy as xp
...

I wonder why import numpy twice as different name?

  1. In function generate_anchor_base():
    py = base_size / 2.
    px = base_size / 2.

Why set the centre of anchors (base_size / 2, base_size / 2)? Isn't (0, 0) more intuitive?

Williamongh avatar Apr 19 '19 07:04 Williamongh

For question2, according to the original paper, I think the author set the anchor point in the center. From the perspective of mine, (0, 0) is also okay because it can propose almost same number of anchor boxes. But what influence it will have on the mAP still needs to be tested.

JacobYuan7 avatar Jul 12 '19 09:07 JacobYuan7

@JacobYuan7 Thank you for your reply. I agree. For Q1, maybe it's just temporary code when debugging.

Williamongh avatar Jul 20 '19 07:07 Williamongh

@JacobYuan7 Thank you for your reply. I agree. For Q1, maybe it's just temporary code when debugging.

Have you tested its influence on mAP about Q2?anyway,(0,0) is more intuitive.

lfxx avatar Jul 14 '20 10:07 lfxx