[pgpointcloud] PDAL error

[Dean.Hand.ctr at nga.mil]

Oh fun.  Glad I could help identify a new issue.  :)

-----Original Message-----
From: Paul Ramsey [mailto:pramsey at cleverelephant.ca] 
Sent: Tuesday, July 08, 2014 2:07 PM
To: howard at hobu.co; Hand Dean NGA-II USA CTR; pgpointcloud at lists.osgeo.org
Cc: Bellenger James -Jim- R Mr NGA-II USA CIV; Terry Theodore B NGA-ID USA CTR; Schwarz Eric L NGA-II USA CIV; Greenlee James -Jim- V NGA-IIG USA CTR
Subject: RE: [pgpointcloud] PDAL error

I’d say it’s now a known issue (I should probably ticket it so we don’t forget) but there’s no patch in play for it.

P.

-- 
Paul Ramsey
http://cleverelephant.ca
http://postgis.net


On July 8, 2014 at 12:04:52 PM, dean.hand.ctr at nga.mil (dean.hand.ctr at nga.mil) wrote:

	
	Howard, 
	
	To clarify, this is a known issue with a version item available as a partial fix? 
	Is there currently a fix to this issue or are we in a holding pattern? 
	
	Dean 
	
	-----Original Message----- 
	From: Howard Butler [mailto:howard at hobu.co] 
	Sent: Tuesday, July 08, 2014 1:58 PM 
	To: Paul Ramsey 
	Cc: Hand Dean NGA-II USA CTR; pgpointcloud at lists.osgeo.org 
	Subject: Re: [pgpointcloud] PDAL error 
	
	
	On Jul 8, 2014, at 1:48 PM, Paul Ramsey <pramsey at cleverelephant.ca> wrote: 
	
	> Looks like libxml is failing to parse the XML document PDAL is generating, which is odd, because the failure is at the basic XML parse level, not at any further level. So this is a particular problem w/ the XML generated (or not) by PDAL, possibly because the API contract there changed? So more of a PDAL problem, perhaps, than a pointcloud one. 
	
	Ah, yes, I did tweak the PDAL schema stuff a little bit recently (last December). I added an orientation to describe whether or not the data are stored dimension-major or point-major. As part of this I mucked with the xmlns [1]. There already is a "version" item available, and it seemed silly to embed and calcify the version as part of the namespace itself. I think pgpointcloud's XML validation needs to be tweaked a little bit to allow this namespace to be valid. 
	
	I think there will need to be a pull request to pgpointcloud to make this change. 
	
	Howard 
	
	[1] https://github.com/PDAL/PDAL/commit/eb5b3d615e99fb2a4e3cd43027ce7f50c25b5743#diff-df9d91f1f874ad990569d91bad6ce31c