-
Notifications
You must be signed in to change notification settings - Fork 122
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
minADE is not right #43
Comments
Decoder.py 174, why you add a 'pass', skip the loss of pred trajs ?causing the complete traj module not trained. |
The 'pass' does not take any effect here. What is the value of |
Sorry,I made a mistake of 'pass'. But 'complete_traj' is indeed True. Can you give me a trained weight? |
These are my parameters, Can you find what is wrong? --argoverse |
It seems that |
It doesn‘t matter,if I don't set model_recover_path, it will go to ./tmp to search the weight file. Can you give me a wechat number for deeper communication for details? |
What is your evaluation command? During evaluation, you should set |
Thank u, I knew my problem. model_recover_path should be set to weights of Set Predictor training, but set_predict-train_recover set should be set to weights of first training. |
when I trained the Set Predictor, eval results are as follows:
other_errors {'stage_one_k': 3.0803698258030754, 'stage_one_recall': 0.9638398086076706, 'set_MR_pred': 0.0738903979863932, 'set_minFDE_pred': 1.3842987156892936}
{'minADE': 14.246385467933312, 'minFDE': 1.3842987156892916, 'MR': 0.0738903979863932}
ADE 14.316873007381048
DE@1 9.875993647573898
DE@2 19.429086244927365
DE@3 3.496420582410066
the minADE is absolutely wrong. I visualize the pred trajs, they are also not right, I think the complete traj module is not trained sufficient, but I trained the model follow your instructions, so where exactly is the problem?
The text was updated successfully, but these errors were encountered: