search

ignacio-rocco / detectorch

Detectorch - detectron for PyTorch
Other
558 stars 72 forks source link

reporting mAP computed for the reproduced models #2

Closed aosokin closed 6 years ago

aosokin commented 6 years ago

Hi, do you plan to add mAP on PASCAL/COCO computed by your code? I suggest adding those to the README directly

ignacio-rocco commented 6 years ago

Hi, I have only tried COCO. I added the mAP to the README!

aosokin commented 6 years ago

Wound't it be better to add original numbers (produced by Caffe2 code) as well to show that you reproduce it well?

ignacio-rocco commented 6 years ago

Hi, maybe it's a bit repetitive as it's the same models running the same operations, so the numbers match exactly.

Le sam. 10 mars 2018 20:30, Anton Osokin notifications@github.com a écrit :

Wound't it be better to add original numbers (produced by Caffe2 code) as well to show that you reproduce it well?

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/ignacio-rocco/detectorch/issues/2#issuecomment-372058525, or mute the thread https://github.com/notifications/unsubscribe-auth/ALa77L4Voa2-jEP1Wnu1o_mJ7QpHNbx5ks5tdCnwgaJpZM4Sky3h .

aosokin commented 6 years ago

@ignacio-rocco it just shows in numbers that you are serious. There are lots of repos online that claim that they reproduce smth, but the numbers are nowhere close. And the numbers are way more concrete compared to saying that in text.