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

About positional encoding of the coordinate-MLP #6

Open
OctoberKat opened this issue Jun 2, 2022 · 1 comment
Open

About positional encoding of the coordinate-MLP #6

OctoberKat opened this issue Jun 2, 2022 · 1 comment

Comments

@OctoberKat
Copy link

Hi, thanks for your great work!
I wonder if you have tried using positional encoding or sine activation schemes in the coordinate-MLP network? It is a very popular method to make the coordinate-MLP fitting high frequencies signals better, but in your source code it seems that the coordinate-MLP is a relu-activated MLP without positional encoding.

@Lilac-Lee
Copy link
Owner

Hi, we have preliminarily tried using positional encoding, which did not help in our case. My understanding is that the scene flow representation for autonomous driving scenes might not necessarily need positional encoding, since we want to make the local region "as rigid as possible". Cheers.

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