<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 21, 2015 at 3:51 AM, Kristian Evers <span dir="ltr"><<a href="mailto:kreve@gst.dk" target="_blank">kreve@gst.dk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div lang="EN-US" link="blue" vlink="purple">
<div>
<p class="MsoNormal">I have noticed a slight inconsistency between the docs and the functionality of las.writer and now wondering which one is wrong.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">On <a href="http://www.pdal.io/stages/writers.las.html" target="_blank">http://www.pdal.io/stages/writers.las.html</a> it says that the default value of the option minor_version is 3, but in reality when you do a ‘pdal translate’ without specifying any
 options you get a las/laz file with format version 1.2. The source code confirms this in line 69 of io\las\LasWriter.cpp:<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal" style="text-indent:36.0pt">m_minorVersion.setDefault(2);<u></u><u></u></p>
<p class="MsoNormal"><span style="font-size:9.0pt;font-family:Consolas;color:#333333;background:white"><u></u> <u></u></span></p>
<p class="MsoNormal">It’s a minor detail, just wondering if the docs or the code needs to be changed. Was it the intention to write las-files with minor_version=3? I’ll do a pull request with a fix once I know if the docs or the code is correct. Personally
 I think that the default minor_version should be 3, as it says in the docs.</p></div></div></blockquote><div><br></div><div>I've fixed the documentation to reflect the code (should show up online shortly).  The default has in reality been version 2 for a long time, so I'm leaving it alone for now, but if there's enough demand, it can certainly be changed.</div><div><br></div><div>Thanks for catching this!</div></div><div><br></div>-- <br><div class="gmail_signature">Andrew Bell<br><a href="mailto:andrew.bell.ia@gmail.com" target="_blank">andrew.bell.ia@gmail.com</a></div>
</div></div>