-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
EfficientNMS plugin doesn't support fp16 #1758
Comments
This seems to be related to an optimization that's in place for sorting in fp16 where it wrongly assumes all scores are within [0-1] range. If your scores are outside of this range, i.e. negative values, it can give wrong results. There will be a fix for this soon. |
@wraveane Does it means must wait until next tensorrt version release? that would be a long time. |
@jinfagang An update to the EfficientNMS plugin has been posted yesterday which, among other things, fixes this FP16 problem on non-sigmoid activated input scores. |
@wraveane thanks. |
you can just +1 for each score in older version of tensorRT , it will very close to original detection score |
I have an onnx model, injected EfficientNMS plugin using graph-surgon, the inference result like this in fp16:
the boxes and labels are normal, but scores are wrong, it was negative values.
using fp32 without this problem.
My env:
The text was updated successfully, but these errors were encountered: