-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Performance on LVIS mAP is low #4
Comments
Hi, thanks for your question! The block you show computes the box AP on LVIS. To get a reasonable number on that, you need to change the NMS setting and test cfg to the default LVIS one which uses intra-class NMS and allows 300 bboxes output. Our current default NMS settings are class agnostic and test cfg only allows outputting a maximum of 50 bboxes per image to optimize the TETA metric, which does not allow the high overlapping predictions in the same location. |
Thanks for replying! test_cfg=dict(
rcnn=dict(
score_thr=0.0001,
nms=dict(type='nms', iou_threshold=0.5, class_agnostic=False, split_thr=100000),
max_per_img=300)
) and gets a more reasonable number of AP. Is there something I might be missing, such as any hyperparameters in test_cfg that might influence the results on tracking? Thank you in advance, great work! |
Thanks for your question! Track mAP by default, you need to as well output lots of overlapping false positive tracks to get a reasonable score. However, the TETer is not designed to output such results. Thus, you have to modify the tracker by yourself to make sure that the tracker still outputs those false positive overlapping tracks. It is strongly recommended that you test the tracker using TETA metric which is designed better analyze the trackers and avoid such false positive overlapping tracks. |
We have reproduced mAP on TAO dataset, and found that the performance is quite low.
The numbers are generated with the block here
, while track3D mAP is calculated with the tao toolkit.
Could you provide the mAP numbers you get on TAO? We need to further verify it there is any bugs here
The text was updated successfully, but these errors were encountered: