Skip to content
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

Bevformer temporal and no temporal configs are the same #6

Open
jasonkhadka opened this issue Jul 27, 2023 · 3 comments
Open

Bevformer temporal and no temporal configs are the same #6

jasonkhadka opened this issue Jul 27, 2023 · 3 comments

Comments

@jasonkhadka
Copy link

Following configs are the same. How did you test non-temporal model?
Temporal config:

zoo/BEVFormer/projects/configs/bevformer/bevformer_base.py

Non-temporal configs:

zoo/BEVFormer/projects/configs/bevformer/bevformer_base_no_temp.py
@Daniel-xsy
Copy link
Owner

I've corrected it. You only need to change the following argument to switch between temporal and non-temporal mode.

video_test_mode=False

Thanks for pointing out this issue.

@jasonkhadka
Copy link
Author

@Daniel-xsy Isn't that only for testing? The training is still done with temporal information with Temporal Attention?

@Daniel-xsy
Copy link
Owner

Yes, that's correct. In the official BEVFormer paper, the authors trained two versions: BEVFormer and BEVFormer-S. The BEVFormer-S is trained without temporal information, i.e., video_test_mode=False for both training and testing. You can find more details here.

However, since we couldn't find the training configuration and checkpoint of BEVFormer-S, we opted to use the BEVFormer checkpoint and turn off video_test_mode during testing. This might cause a discrepancy between training and testing, as evident from the lower performance of BEVFormer-non-temporal compared to the official results. We might try to reproduce this version and update the checkpoint in the future if possible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants