search

Gradiant / pyodi

Python Object Detection Insights
https://gradiant.github.io/pyodi/
Mozilla Public License 2.0
192 stars 16 forks source link

Remove `anchor_base_sizes` from `train_config_generation` #67

Closed daavoo closed 3 years ago

daavoo commented 4 years ago

anchor_base_sizes is None by default in mmdet. The same anchor distributions can be generated without this argument.

github-actions[bot] commented 4 years ago

Branch issue-67-Remove_anchor_base_sizes_from_train_config_generation created!

igonro commented 4 years ago

@daavoo this issue is already done? I saw that you have commited to the branch, is it ready to open a pull request to master?

daavoo commented 4 years ago

I think that I stopped working on the specific issue because a broader question about the train_config_generation app arised. We have to properly benchmark the utility of this tool and clarify why and how it works, there are many uncertain points like the base_sizes, the strategy for assigning boxes to anchor levels, etc.

mmeendez8 commented 4 years ago

Yes, we discussed about this, anchor_base_sizes is necessary if you want to create specific configuration of anchors. It is not necessary when ratios between anchors sizes are constant (4,8,16,32) but it can be useful to create different ones like (6,8,32,64)