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.

Sign up for GitHub

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

Jump to bottom

Adding glomap support to ns-process-data, works with COLMAP 3.8! #3409

Open
wants to merge 16 commits into
base: main
Choose a base branch
from
Open

Adding glomap support to ns-process-data, works with COLMAP 3.8! #3409

wants to merge 16 commits into from
+54 −8

Conversation

AntonioMacaronio
Copy link
Contributor

@AntonioMacaronio AntonioMacaronio commented Sep 4, 2024 •
edited
Loading

Problems and Background

  • GLOMAP is orders of magnitudes faster than COLMAP. On average, it finds camera extrinsics, intrinsics, and other distortion parameters and such 100x faster than COLMAP, and it would great to support this in Nerfstudio and already has been requested

Overview of Changes

  • Adding a GLOMAP toggle in ns-process-data script, which uses the GLOMAP mapper
  • Updated the loading messages

@kerrj
Copy link
Collaborator

kerrj commented Sep 4, 2024 •
edited
Loading

This is great! It would also be good to write documentation or link to installation instructions and the like

@AntonioMacaronio AntonioMacaronio changed the title Adding glomap support to ns-process-data, requires latest version of COLMAP Adding glomap support to ns-process-data, works with COLMAP 3.8! Sep 6, 2024
@AntonioMacaronio AntonioMacaronio marked this pull request as ready for review September 10, 2024 04:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pablovela5620 pablovela5620 pablovela5620 left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@AntonioMacaronio @kerrj @pablovela5620