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

Limit initial zoom to far_plane #40

Merged
merged 1 commit into from
Nov 22, 2021
Merged

Limit initial zoom to far_plane #40

merged 1 commit into from
Nov 22, 2021

Conversation

ch-sa
Copy link
Owner

@ch-sa ch-sa commented Nov 22, 2021

  • take minimum of diagonal and 90% far_plane
  • assume that there are at least some points from the center visible

Closes: #39
Relates: #36

 - take minimum of diagonal and 0.9 * far_plane

 Closes: #39
 Relates: #36
@ch-sa ch-sa merged commit cea9cd5 into master Nov 22, 2021
@ch-sa ch-sa deleted the ch-sa/initial-zoom branch November 22, 2021 20:52
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

Successfully merging this pull request may close these issues.

pcd point cloud not shown
1 participant