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

This link is no longer valid #31

Open
Dongdong-NEU opened this issue Jun 12, 2024 · 3 comments
Open

This link is no longer valid #31

Dongdong-NEU opened this issue Jun 12, 2024 · 3 comments

Comments

@Dongdong-NEU
Copy link

Thank you for your outstanding open-source work.

I have a few questions:

This link is no longer valid: https://faubox.rrze.uni-erlangen.de/dl/fiUNWMmsaEAavXHfjqxfyXU9/door.zip.
colmap_wrapper==1.1.5 is not compatible with the aruco-estimator project.

@meyerls
Copy link
Owner

meyerls commented Jun 12, 2024

The link should now work again!

Unfortunately I have no quick answer for the colmap wrapper. My first guess would be to install colmap wrapper from source

@Dongdong-NEU
Copy link
Author

The link should now work again!

Unfortunately I have no quick answer for the colmap wrapper. My first guess would be to install colmap wrapper from source

Thank you for your prompt reply. I have a small question: Is this the correct dataset? The mapping results don't quite match those described on the GitHub introduction page.
image

@meyerls
Copy link
Owner

meyerls commented Jun 12, 2024

For the frustum you could decrease the size here: vis.visualization(frustum_scale=0.7, point_size=0.1).

And for the missing dense point cloud you can specify it while loading the colmap project. I guess this broke while modifying colmap wrapper

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