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

gen_sineembed_for_position #17

Open
rolyeMMlin opened this issue Jan 21, 2025 · 8 comments
Open

gen_sineembed_for_position #17

rolyeMMlin opened this issue Jan 21, 2025 · 8 comments

Comments

@rolyeMMlin
Copy link

why exchange ref points pos coordinate, input is [x, y],but return [y, x]?

@fishmarch
Copy link
Contributor

I remember these codes were copied from DAB-DETR. I also noticed this, but its influence seemed to be minimal.

@rolyeMMlin
Copy link
Author

i think obstacle be less affected, but its influence the lane or edge,beacuse the reference point is relatively long

@fishmarch
Copy link
Contributor

I remember I tried to change the codes, there was no clear influence. The network could learn to know this.

@rolyeMMlin
Copy link
Author

i try to add this sineembed_for_position,but KPI are consistent compared with no sineembed_for_position,its my problem of my realization?

@fishmarch
Copy link
Contributor

Where did you add this sineembed? This embedding has been used in our codes.

@rolyeMMlin
Copy link
Author

sin cos embedding it's hard to deploy on J6,so we removed sineembed,add reference points with instance query by a linear

@rolyeMMlin
Copy link
Author

pose embedding influence is remarkable?

@fishmarch
Copy link
Contributor

I did not try other types of embedding...other embeddings should also have some effect.

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