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

las 1.4 extended classifications #445

Open
jjrise opened this issue Sep 8, 2020 · 5 comments
Open

las 1.4 extended classifications #445

jjrise opened this issue Sep 8, 2020 · 5 comments

Comments

@jjrise
Copy link

jjrise commented Sep 8, 2020

this may be something I'm overlooking, but I'm having a tough time getting extended classifications to convert or view. These originate from a las 1.4 file initially saved in terrasolid. These are not "extra bytes" las 1.2 classifications (i.e. terrascan), but rather 64-255 user defined classifications.

Any help would be appreciated!

@m-schuetz
Copy link
Collaborator

If they're just the normal 0-255 byte classification, take a look at the classification example: http://potree.org/potree/examples/classifications.html
Use setTreeScheme() as a template to define your own color scheme.

@jjrise
Copy link
Author

jjrise commented Sep 9, 2020

Hi Markus, I did notice this, however it seems my classifications from my input (las 1.4 extended classes) do not properly convert through PotreeConverter 1.7. I'm using the classification example with setTreeScheme(), which is working as expected, but those classes do not hold any points after conversion.

EDIT: I should mention that the points that originated in extended classes are not missing, but they are all in the Default class after conversion.

@m-schuetz
Copy link
Collaborator

Can you send me an example las file?

@jjrise
Copy link
Author

jjrise commented Sep 9, 2020

Can you send me an example las file?

https://drive.google.com/file/d/1lwnlgSMajbZTGZKh7g9-hd1EjZ2FSOeA/view?usp=sharing

@m-schuetz I gave permissions to your email. Let me know if that doesn't work for you.

@arknoll
Copy link

arknoll commented Mar 18, 2021

@m-schuetz Checking in on this topic. I too have a las file with extended classes. They are all getting smooshed down into the first 31 classes.

m-schuetz added a commit that referenced this issue Jun 2, 2022
Handle extended classifications. Issue #445
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

3 participants