search

mfbonfigli / gocesiumtiler

A Cesium.js point cloud 3D tiles generator from LAS files written in Golang
Mozilla Public License 2.0
195 stars 40 forks source link

Las 1.4 compatibility for 0-255 classifications #49

Open mfbonfigli opened 3 months ago

mfbonfigli commented 3 months ago

Opening this issue to verify the compatibility issue between Las 1.4 and classifications 0-255 as per the comment https://github.com/mfbonfigli/gocesiumtiler/issues/16#issuecomment-2243307800

I just tested it this morning and las 1.4 does appear to be working. However, it seems to only include 0-31 classifications instead of the full 0-255 classifications in 1.4. Is there a flag or way to do this that I'm missing?

mfbonfigli commented 1 week ago

I can confirm the bug, this happens for point formats 6+ as the code fails to take into account that in the specs for those point types the classification flags have been moved out of the classification byte and hence the full byte is available to be used to store the class value.

This will be fixed soon in the next release.