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

Fix bug where chunks containing exactly 10000 points are lost during indexing #607

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

kallus
Copy link

@kallus kallus commented Feb 1, 2023

This is most likely the cause behind issue #597

Before this fix, the created octree.bin is 10000 * bits_per_point bytes smaller than it should be according to the point count and bits per point in the created metadata.json. Thus, the resulting potree point cloud contains 10000 points less than the input las file and than what it says in the metadata.json file.

When the indexer builds the hierarchy, this bug caused recursion to stop when encountering a chunk containing exactly 10000 points, neither recursing down into parts of the chunk nor writing the chunk out to the octree.bin file.

Unfortunately I can not publish the las input files that are affected by this bug.

@m-schuetz m-schuetz added the bug label May 16, 2023
petern-annotell pushed a commit to petern-annotell/PotreeConverter that referenced this pull request Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants