Opened 5 years ago
Closed 3 years ago
#3924 closed defect (fixed)
Port r.in.lidar to PDAL
| Reported by: | marisn | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 8.0.0 |
| Component: | Raster | Version: | svn-trunk |
| Keywords: | lblas, pdal, r.in.lidar | Cc: | |
| CPU: | Unspecified | Platform: | Unspecified |
Description
As libLAS is more/less EOL, we need a PDAL based replacement. At the moment there is an extension r.in.pdal, but 1) it is extension and not a core module 2) it uses PDAL CLI instead of working directly with API 3) it is not feature complete replacement of r.in.lidar.
IMHO we could try to IFDEF libLAS/PDAL code in r.in.lidar and make PDAL/LibLAS exclusive options in configure. Thus transition from libLAS to PDAL would be seamless for ordinary r.in.lidar users.
I would love to see any progress on it as I got hit by an old issue returning on Ubuntu Disco with libLAS C API code resulting in a segfault and thus far I haven't been able to figure out how to work around it. https://lists.osgeo.org/pipermail/grass-dev/2014-May/069046.html
Change History (7)
comment:1 by , 5 years ago
comment:3 by , 5 years ago
No, I missed it. I apologize for my inattention.
My contrition, however, is somewhat tempered by my happiness :-)
Doug
comment:6 by , 5 years ago
| Milestone: | → 7.8.3 |
|---|
comment:7 by , 3 years ago
| Milestone: | 7.8.3 → 8.0.0 |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
r.in.pdal will land 8.0 after merge of PR https://github.com/OSGeo/grass/pull/1200
My solution for 14.04 and 16.04 was to compile liblas from source. That being said, I agree on the need to move to pdal.
Not a C programmer, but willing to compile on the home computer, break things, and send reports back.