<div dir="ltr"><br><br>2018-08-06 12:12 GMT+02:00 Anika Bettge <<a href="mailto:bettge@mundialis.de">bettge@mundialis.de</a>>:<br>><br>> Hi all,<br>><br>> I just uploaded my first GRASS GIS addon r.in.pdal:<br>><br>> <a href="https://trac.osgeo.org/grass/browser/grass-addons/grass7/raster/r.in.pdal">https://trac.osgeo.org/grass/browser/grass-addons/grass7/raster/r.in.pdal</a>.<br><div><br></div><div>Thanks a lot to your contribution Anika!</div><div><br></div><div>A few cosmetic suggestions:</div><div><br></div><div>the print command in L268 should be replaced with grass.message because this is not shell-style output</div><div>L353:</div><div>         grass.message(_("Generating output raster map <%s>...") % outfile)<br></div><div>seems to be at the wrong place because nothing happens after that message</div><div>same for L191</div><div><br></div><div>you can you set the metadata on the resultant output raster with r.support, otherwise metadata will be set by <a href="http://r.in.xyz">r.in.xyz</a> (the module that actually created the raster)<br></div><div><br></div><div>there are some comments on gdalwarp, but gdalwarp is not used by r.in.pdal</div><div><br></div><div>Best,</div><div><br></div><div>Markus M</div><div><br></div><div>></div>><br>> Best regards,<br>> Anika<br>><br>> --<br>>  <br>>   Anika Bettge<br>>   - Anwendungsentwicklerin -<br>><br>>   mundialis GmbH & Co. KG<br>>   Kölnstraße 99<br>>   53111 Bonn<br>><br>>   Tel: +49 (0)228 / 38 75 80 -80<br>>   Fax: +49 (0)228 / 96 28 99 -57<br>><br>>   Email: <a href="mailto:bettge@mundialis.de">bettge@mundialis.de</a><br>>   Web: <a href="https://www.mundialis.de">https://www.mundialis.de</a><br>><br>>   Amtsgericht Bonn, HRA 8528<br>>   Komplementärin: mundialis Verwaltungsgesellschaft mbH<br>>   vertreten durch: Dr. Markus Neteler, Hinrich Paulsen, Till Adams<br>>   <br>>   Informationen über Ihre gespeicherten Daten finden Sie auf unserer Homepage unter folgendem Link:<br>>   <a href="https://www.mundialis.de/datenschutzerklaerung">https://www.mundialis.de/datenschutzerklaerung</a><br>><br>> _______________________________________________<br>> grass-dev mailing list<br>> <a href="mailto:grass-dev@lists.osgeo.org">grass-dev@lists.osgeo.org</a><br>> <a href="https://lists.osgeo.org/mailman/listinfo/grass-dev">https://lists.osgeo.org/mailman/listinfo/grass-dev</a><br><br></div>