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

Some details to discuss #10

Open
Liaoqing-up opened this issue Nov 8, 2022 · 1 comment
Open

Some details to discuss #10

Liaoqing-up opened this issue Nov 8, 2022 · 1 comment

Comments

@Liaoqing-up
Copy link

Thank you for open-sourcing your work. I was wondering, why you use x_up(the current frame's bev feature) other than x_up_fuse(the sequential frames through spatial-aware fusion) as center query embedding ? Apologies if I missed it in the paper.

@Liaoqing-up Liaoqing-up changed the title Some details to be discuss Some details to discuss Nov 8, 2022
@edwardzhou130
Copy link
Collaborator

Sorry for the late reply. There is no empirical reason for me to choose x_up rather than x_up_fuse. Center classification and box regression need two different types of information from previous frames, so I want to avoid mixing them.

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