[pdal] Problem reading LAZ file

[Howard Butler]

This is actually my fault. 

PDAL and libLAS currently use the LASzipper/LASunzipper interface, but everyone else who integrates LASzip uses the "DLL" interface. Going forward there will be no more maintenance or improvement of the zipper/unzipper interface. PDAL and libLAS need to change to use this new interface, especially to catch upcoming LAZ 1.4 support. It hasn't been done yet.

I updated the LASzip master tree to build and link the "DLL" interface on *nix, and in the process I removed the zipper/unzipper interface. This means that PDAL/libLAS need to stick to released LASzip. There have been no functional updates to the LASzip codebase since the last release of it.

The LASzip distribution at http://github.com/LASzip/LASzip is the LAZ software that Martin throws over the wall into a single library for others to implement LAZ support in their software. It lags LAStools' capability a little bit, but there have been no wire format changes in a very long time.

Howard

> On Mar 30, 2016, at 11:56 AM, Andrew Bell <andrew.bell.ia at gmail.com> wrote:
> 
> Trying to figure out if this is something to fix in PDAL or LASzip and if the latter, how to get it done.
> 
> On Wed, Mar 30, 2016 at 12:27 PM, Andrew Bell <andrew.bell.ia at gmail.com> wrote:
> Can anyone explain the relationship between the Github repo and Martin's distribution?
> 
> Is there some reason that a breaking change was made to LASzip?  Seems that one could have had the zipper/unzipper point at laszip_api.h
> 
> On Wed, Mar 30, 2016 at 9:59 AM, Helimap Postmaster <postmaster at helimap.ch> wrote:
> That works, thanks!
> 
> On Wed, Mar 30, 2016 at 4:49 PM, Helimap Postmaster <postmaster at helimap.ch> wrote:
> Thanks Todd,
> 
> I saw your answer while writing thecomment, will try with e7065cb.
> 
> On Wed, Mar 30, 2016 at 4:45 PM, Murphy, Todd <ROBERT.T.MURPHY at leidos.com>wrote:
> 1. Unsure why it did not pickup up include directory. Have you tried setting it manually? On my Fedora install I cloned the laszip repo from Howards GitHub site (https://github.com/LASzip/LASzip). It installed under /usr/include and /usr/lib.
> 
>  
> 
> 2.I had an issue with the latest laszip which threw the similar errors you identified with LASzipper and LASunzipper. Recommend using earlier commit (e7065cb) of Laszip. My email discussion with Howard on this follows:
> 
>  
> 
> > 2. I just created a new development environment (Fedora 22) using the latest laszip from that repo. I saw you had you had the commit to remove LASzipper and LASunzipper. It appears some of the tests still reference these as well as the lasdump apps in PDAL. I haven’t had time to look into, but did a work around for now. Just wanted you to be aware.
> 
>  
> 
> LASzip needs to be pinned to a release version for now. Martin wanted to remove LASzipper/LASunzipper and use his "DLL" interface. We're starting on LAZ 1.4, and I want to get things in place for PDAL to support all of it once that's ready. I started doing interface stuff based on his new DLL thingy, but PDAL/libLAS aren't yet caught up to it. PDAL's docker image is pinned to e7065cbc5bdbbe0c6e50c9d93d1cd346e9be6778 which should be perfectly good going forward until PDAL catches up to LASzip.
> 
>  
> 
> Todd
> 
>  
> 
> From: pdal [mailto:pdal-bounces at lists.osgeo.org] On Behalf Of Helimap Postmaster
> Sent: Wednesday, March 30, 2016 10:24 AM
> To: pdal
> Subject: [pdal] Problem reading LAZ file
> 
>  
> 
> The error message:
> PDAL: Can't read compressed file without LASzip or LAZperf decompression library.
> 
> I've, however, activated the option WITH_LASZIP. I've also cloned the LASZIP repository and installed it. Indeed, pdal ccmake has changed the library field:
> 
> LASZIP_INCLUDE_DIR               LASZIP_INCLUDE_DIR-NOTFOUND                                                       
> LASZIP_LIBRARY                   /usr/local/lib/liblaszip.so
> 
> Any ideas?
> 
> I've also tried using https://launchpad.net/~pdal/+archive/ubuntu/travis , but that one led to compilation errors. (Dumper.cpp: undefined reference to `LASunzipper::LASunzipper()' )
> 
> Cheers,
> 
> Pol
> 
>  
> 
> 
> 
> 
> _______________________________________________
> pdal mailing list
> pdal at lists.osgeo.org
> http://lists.osgeo.org/mailman/listinfo/pdal
> 
> 
> 
> -- 
> Andrew Bell
> andrew.bell.ia at gmail.com
> 
> 
> 
> -- 
> Andrew Bell
> andrew.bell.ia at gmail.com
> _______________________________________________
> pdal mailing list
> pdal at lists.osgeo.org
> http://lists.osgeo.org/mailman/listinfo/pdal