-
Notifications
You must be signed in to change notification settings - Fork 105
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
Need for NMS #126
Comments
No you are not missing anything. You just need to train more (if you have the enough data of course). PS: Im not the author, just some random enthusiast |
Hey Amir, I can confirm that I have sufficient data and also the losses do look saturated. So training further makes doesn't seem like an option.
|
Are you fine tuning or training from scratch? if not fine tuning, do that instead. Moreover, Just because the loss becomes saturated it doesnt mean the model have converged well enough. Maybe you need to a bit of hyper parameter tuning as well. |
HI,
I realise that the model is based on DETR and uses the non-NMS approach. But after training my model and inferring at 0.3 detection threshold, I get multiple boxes.
But if I increase the threshold to 0.5, I do see fewer boxes but there are multiple boxes still left.
Increasing the threshold further only impacts other classes and I get overall lower detection boxes.
Am I missing something? What is the correct way to tackle this for such non-NMS based models?
The text was updated successfully, but these errors were encountered: