<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>
</head>
<body dir="ltr">
<div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Helvetica,sans-serif;" dir="ltr">
<p>Looking at <a href="https://trac.osgeo.org/gdal/browser/trunk/gdal/frmts/openjpeg/openjpegdataset.cpp?rev=40418" class="OWAAutoLink" id="LPlnk16710" previewremoved="true">https://trac.osgeo.org/gdal/browser/trunk/gdal/frmts/openjpeg/openjpegdataset.cpp?rev=40418</a></p>
<p><br>
</p>
<p>I now see the GDAL made changes to enable multi-threading.</p>
<p><br>
</p>
<p>Thanks</p>
<p>Noureddine Farah</p>
</div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> N. Farah <nfarah@hotmail.com><br>
<b>Sent:</b> Tuesday, October 31, 2017 3:50:15 PM<br>
<b>To:</b> Even Rouault; gdal-dev@lists.osgeo.org<br>
<b>Subject:</b> Re: [gdal-dev] Open Jpeg 2.3.0 and OPJ_NUM_THREADS</font>
<div> </div>
</div>
<style type="text/css" style="display:none">
<!--
p
        {margin-top:0;
        margin-bottom:0}
-->
</style>
<div dir="ltr">
<div id="x_divtagdefaultwrapper" dir="ltr" style="font-size:12pt; color:#000000; font-family:Calibri,Helvetica,sans-serif">
<p>Thanks for the quick response. I'm not using latest GDAL and was curious where that multi-threading enabling was made ? Was it some where in GDAL code or through an option in the openjpeg 2.3.0 API ?</p>
<p><br>
</p>
<p>Thanks</p>
<p>Noureddine Farah</p>
</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>From:</b> Even Rouault <even.rouault@spatialys.com><br>
<b>Sent:</b> Tuesday, October 31, 2017 3:47:01 PM<br>
<b>To:</b> gdal-dev@lists.osgeo.org<br>
<b>Cc:</b> N. Farah<br>
<b>Subject:</b> Re: [gdal-dev] Open Jpeg 2.3.0 and OPJ_NUM_THREADS</font>
<div> </div>
</div>
<style type="text/css">
<!--
p, li
        {white-space:pre-wrap}
-->
</style>
<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">
On mardi 31 octobre 2017 19:39:23 CET N. Farah wrote:</p>
<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">
> </p>
<p style="margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; text-indent:0px">
> In http://gdal.org/frmt_jp2openjpeg.html under 'Thread Support' section</p>
<p style="margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; text-indent:0px">
> there is the following:</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">
> "Starting with GDAL 2.3, this multi-threading at code-block level is</p>
<p style="margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; text-indent:0px">
> automatically enabled by GDAL".</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">
> Was meant "Starting with OpenJpeg 2.3, ..." ?</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">
No, was meant starting with GDAL 2.3 (the current dev version). And that requires openjpeg >= 2.2 as well.</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">
> If yes, was that enabling made in the 'OpenJpeg 2.3' library or was made in</p>
<p style="margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; text-indent:0px">
> GDAL ?</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">
openjpeg >= 2.2 (actually 2.1.2, but I restricted to 2.2 or later) brings multithreading decoding at codeblock level, and GDAL trunk automatically enables it.</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 the doc is correct I think, but I know that those similar version numbers between GDAL and openjpeg can be confusing.</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">
Bottom line: use the latest and greatest of both to get the better performance.</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">
-- </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">
http://www.spatialys.com</p>
</div>
</div>
</body>
</html>