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

The Difference Between "forward_symmetric" and "forward" #30

Open
fighterzzzh opened this issue Apr 11, 2024 · 4 comments
Open

The Difference Between "forward_symmetric" and "forward" #30

fighterzzzh opened this issue Apr 11, 2024 · 4 comments

Comments

@fighterzzzh
Copy link

Hello, great job! I have a question to ask. What is the difference between "forward_symmetric" and "forward"? Looking forward to your reply.

@Parskatt
Copy link
Owner

Hi. The dense warp we compute is unidirectional, and therefore results are asymmetric depnwnding on order of A,B.

We can improve results somewhat by running the decoding and refinement twice (AB, BA). This is more expensive but can be useful in some cases. This is called symmetric.

@fighterzzzh
Copy link
Author

Hi. The dense warp we compute is unidirectional, and therefore results are asymmetric depnwnding on order of A,B.

We can improve results somewhat by running the decoding and refinement twice (AB, BA). This is more expensive but can be useful in some cases. This is called symmetric.

Thank you for your answer. I noticed there are two instances of forward inference in the code. Is there any connection with forward_symmetry。

@fighterzzzh
Copy link
Author

Hi. The dense warp we compute is unidirectional, and therefore results are asymmetric depnwnding on order of A,B.

We can improve results somewhat by running the decoding and refinement twice (AB, BA). This is more expensive but can be useful in some cases. This is called symmetric.

Additionally, in the outdoor model's matching, the entire module underwent two rounds of inference. The second round of inference included certainty and flow

@Parskatt
Copy link
Owner

Yes, or rather the refinement stage is repeated but in higher resolution. The is the reason for "upsample_res"

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