8000 Support visualization of models with arbitrary origin and scale by ahojnnes · Pull Request #3044 · colmap/colmap · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Support visualization of models with arbitrary origin and scale #3044

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
Merged
merged 17 commits into from
Jun 30, 2025
Merged

Support visualization of models with arbitrary origin and scale #3044

merged 17 commits into from
Jun 30, 2025
+189 −138

Conversation

ahojnnes
Copy link
Contributor
@ahojnnes ahojnnes commented Dec 15, 2024 •
edited
Loading

Normalizes the reconstruction purely in model viewer space. In support of GPS aligned models with large coordinates.

@ahojnnes ahojnnes mentioned this pull request Dec 15, 2024
Draft
@StonerLing
Copy link
Contributor

Thanks. It solves the issue of visualizing models with large coordinates in my test.

A small issue: the View center grid and Coordinate axes will move away from the model origin when I try to rotate, translate or zoom the model.

@ahojnnes
Copy link
Contributor Author

Thanks. It solves the issue of visualizing models with large coordinates in my test.

A small issue: the View center grid and Coordinate axes will move away from the model origin when I try to rotate, translate or zoom the model.

This should be fixed now.

@ahojnnes ahojnnes enabled auto-merge (squash) June 30, 2025 10:51
@ahojnnes ahojnnes merged commit 00574a2 into main Jun 30, 2025
14 checks passed
@ahojnnes ahojnnes deleted the user/jsch/model-viewer-scale branch June 30, 2025 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@B1ueber2y B1ueber2y B1ueber2y approved these changes

@paulinus paulinus Awaiting requested review from paulinus

@sarlinpe sarlinpe Awaiting requested review from sarlinpe

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
@ahojnnes @StonerLing @B1ueber2y
0