<div dir="ltr">I'm pretty much up to head in production and testing is doing fine.  But... It's only been a couple hours.  If there is nothing from me in the next day or two, I give it a +1.  Thanks for the hard work!<div><br></div><div>My first pass check with TDLPACK is that I know of nobody who uses it.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 4, 2017 at 11:46 AM, Even Rouault <span dir="ltr"><<a href="mailto:even.rouault@spatialys.com" target="_blank">even.rouault@spatialys.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><u></u>
<div style="font-family:'Sans Serif';font-size:9pt;font-weight:400;font-style:normal">
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">Hi,</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">I wanted to inform you that the GRIB driver has received significant updates lately in trunk :</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">* the underlying degrib and g2clib libraries used to do the GRIB1 & GRIB2 parsing have been updated to their latest upstream versions (*). Given that they are patched in GDAL for various reasons (virtual I/O capabilities, security fixes, other fixes), the update involved merging, and thus regressions might potentially have sliped in.</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">* various improvements on top of that:</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">   - faster gdalinfo on GRIB2 products on multiband datasets (no need to decompress the grid data of the band other than the first one)</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">   - avoid erroring out on unsupported product definition template</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">   - fix in the GRIB1 reader that should make it possible to read more products (those with NV,the number of vertical coordinate parameters, different from 0, which causes previously error messages like "GRIB1 GDS: Expect PV = 255 !=33")</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">* addition of GRIB2 write support</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">So I'd appreciate testing from users to confirm if products they managed to read with past versions are still readable with trunk.</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">I'm wondering if anyone relies on the support for the TDLPACK that apparently the degrib library supports. This isn't even documented in the</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"><a href="http://gdal.org/frmt_grib.html" target="_blank">http://gdal.org/frmt_grib.html</a> help page.  From what I can read in </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"><a href="https://github.com/eengl/pytdlpack" target="_blank">https://github.com/eengl/<wbr>pytdlpack</a> , it seems that TDLPACK is mostly an internal format of NOAA/NWS Meteorological Development Lab. Couldn't find any publicly available file in that format. If nobody uses this capability, I'd prefer we comment out all the parts related to TDLPack so as to reduce the vulnerability surface.</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">Even</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">(*) <a href="https://svn.osgeo.org/gdal/trunk/gdal/frmts/grib/degrib/README.TXT" target="_blank">https://svn.osgeo.org/gdal/<wbr>trunk/gdal/frmts/grib/degrib/<wbr>README.TXT</a></p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">-- </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">Spatialys - Geospatial professional services</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"><a href="http://www.spatialys.com" target="_blank">http://www.spatialys.com</a></p></div><br>______________________________<wbr>_________________<br>
gdal-dev mailing list<br>
<a href="mailto:gdal-dev@lists.osgeo.org">gdal-dev@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/gdal-dev" rel="noreferrer" target="_blank">https://lists.osgeo.org/<wbr>mailman/listinfo/gdal-dev</a><br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">--<div><a href="http://schwehr.org" target="_blank">http://schwehr.org</a></div></div>
</div>