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

Questions about the parameter add_halfpix. #81

Open
xulongbai opened this issue Jun 29, 2024 · 2 comments
Open

Questions about the parameter add_halfpix. #81

xulongbai opened this issue Jun 29, 2024 · 2 comments

Comments

@xulongbai
Copy link

Hi, thanks for the great work!

I have two questions about the parameter add_halfpix.

  • Why is add_halfpix set to False in the Hypersim dataset but True in the Tanks and Temples dataset? Both datasets seem to use the DeepLSD line detector by default.
  • Is it necessary to add half a pixel for the vanishing point?

Looking forward to your reply :)

@B1ueber2y
Copy link
Member

The option is more for the convention to define pixel coordinates (to be at the corner or at the center of the pixel), where in COLMAP convention there is generally an half pixel offset while in Hypersim it is not. Empirically this does not make a huge difference in my experience.

@xulongbai xulongbai closed this as not planned Won't fix, can't repro, duplicate, stale Dec 16, 2024
@xulongbai
Copy link
Author

Hi Shaohui,

Thank you for your thoughtful response. After reviewing the code, I tentatively believe that adding half a pixel may not pertain to the dataset itself but rather to the coordinate system used for the detected 2D line segments. This observation holds true for both the Hypersim and Tanks and Temples datasets, as they rely on COLMAP's camera model within LIMAP.

Regarding the coordinate system for 2D line segments, I reached out to Rémi Pautrat, the first author of DeepLSD, via email some time ago. Rémi, who contributed to the development of both DeepLSD (the deeplsd detector in LIMAP) and pytlsd (the lsd detector in LIMAP), clarified that the coordinate origins for 2D line segments detected by both DeepLSD and pytlsd are located at the top-left corner of the top-left pixel in the image. This convention aligns with the COLMAP coordinate system. I am deeply grateful to Rémi Pautrat for his assistance.

Based on this, I believe that using DeepLSD and pytlsd with any dataset should not necessitate adding half a pixel.

I eagerly await your thoughts on this matter. Thank you once again for your time and expertise! (And my apologies for inadvertently closing the issue.)

Best,
Xulong Bai

@xulongbai xulongbai reopened this Dec 16, 2024
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