<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Sat, Sep 26, 2015 at 8:40 PM, Vaclav Petras <span dir="ltr"><<a href="mailto:wenzeslaus@gmail.com" target="_blank">wenzeslaus@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Dear all,<br><br></div>in r66343 I've added the option to store return or class as category. This is much faster than storing them as attributes. Also, the further processing will be faster supposing that the subsequent workflow can use categories in the same way as attributes.<br><br>Return and class can be stored at the same time, each in its own layer, otherwise they would get mixed. There is no protection against specifying same layer for all, perhaps there should be.<br></div></blockquote><div><br></div><div>Another thing which needs further consideration is category number when class number is 0 (ASPRS: Created, never classified). Is 0 allowed for category? Library documentation says no [1] but then it says yes for OGR layers (and I suppose PostGIS as well). So how hard is the requirement? If 0 should be changed in case of ASPRS/lidar class 0, then to what?<br><br>[1] <a href="https://grass.osgeo.org/programming7/vectorlib.html#vlibCategoriesLayers">https://grass.osgeo.org/programming7/vectorlib.html#vlibCategoriesLayers</a><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div><br></div><div>Please note that this change lacks test and documentation. The same still applies to two recent changes in r.in.lidar and v.in.lidar.<br></div><div><br></div><div>Vaclav<br><br></div><div><br>v.in.lidar: store return and class as cats or store no cats<br><a href="https://trac.osgeo.org/grass/changeset/66343" target="_blank">https://trac.osgeo.org/grass/changeset/66343</a><br><br>v.in.lidar: use unsigned long long for counting point counts<br><a href="https://trac.osgeo.org/grass/changeset/66255" target="_blank">https://trac.osgeo.org/grass/changeset/66255</a><br><br>v.in.lidar: decimation (skip, preserve, offset, limit)<br><a href="https://lists.osgeo.org/pipermail/grass-dev/2015-September/076275.html" target="_blank">https://lists.osgeo.org/pipermail/grass-dev/2015-September/076275.html</a><br><br>r.in.lidar: height above ground<br><a href="https://lists.osgeo.org/pipermail/grass-dev/2015-September/076147.html" target="_blank">https://lists.osgeo.org/pipermail/grass-dev/2015-September/076147.html</a></div></div></div>
</blockquote></div><br></div></div>