Opened 5 years ago

Closed 3 years ago

#3924 closed defect (fixed)

Port r.in.lidar to PDAL

Reported by: marisn Owned by: grass-dev@…
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 dnewcomb, 5 years ago

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.

comment:2 by neteler, 5 years ago

Did you see this?

r.in.pdal - PDAL-based version of r.in.lidar

comment:3 by dnewcomb, 5 years ago

No, I missed it. I apologize for my inattention.

My contrition, however, is somewhat tempered by my happiness :-)

Doug

comment:4 by neteler, 5 years ago

Milestone: 7.8.17.8.2

Ticket retargeted after milestone closed

comment:5 by neteler, 5 years ago

Milestone: 7.8.2

Ticket retargeted after milestone closed

comment:6 by neteler, 5 years ago

Milestone: 7.8.3

comment:7 by marisn, 3 years ago

Milestone: 7.8.38.0.0
Resolution: fixed
Status: newclosed

r.in.pdal will land 8.0 after merge of PR https://github.com/OSGeo/grass/pull/1200

Note: See TracTickets for help on using tickets.