-
Notifications
You must be signed in to change notification settings - Fork 78
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
dt_annos and gt_annos params in the calculate_iou_partly
function are swapped.
#25
Comments
I rerun a freshly cloned repo but the result is fine. It takes about 6 hours on my 1080 Ti with SSD. Could you provide me the tensorboard log? loss config and more.
|
Thanks for the prompt reply. Here is the loss screenshot and the tensorboard log file - LOSSTensorboard log |
What I can directly identified is that both of my loss goes down much faster than yours. And according to the tensorboard log, the recorded model structure is the same, and you have only changed the path and slightly increase the epoch number (which is totally fine). events.out.tfevents.1626880244.yxliu-ramlab.10362.0.log I can not diagnose this problem. Here is something you can try:
By the way, calculate_iou_partly is basically borrowed from other repos and I did not modify the detail. Maybe it is a inherited harmless "bug". |
OK. Here are some more evaluations. Pretrained model provided in the release evaluated on the Validation set.
Model trained and evaluated on the debug split.
|
When using the precompute results downloaded in the release, my result is:
When using the original precompute result on chen's split the result is exactly the same as yours.
The debug split result is rather bad. For me I can go to 50 - 60 mAP. It is clear that it is the training process that goes wrong? |
OK. I'll try to fix it. This helps a lot. Thank you :) |
Hi Owen,
Great work! Thanks for uploading the code and providing very clear instructions to run it.
I have two issues that I wanted to ask -
First, I noticed that at the above line dt_annos and gt_annos params in the
calculate_iou_partly
function are swapped. I am not sure if it matters because IoU operation is commutative.Second, I ran training for Mono3D with the example config provided in the repo. I was trying to reproduce the results but I am always getting results which are not similar to the expected results on the validation set.
Here are my results.
Here are my training steps -
Can you please tell me what I am doing wrong here?
Thanks!
The text was updated successfully, but these errors were encountered: