<div dir="ltr">Hello,<br><b style="font-weight:normal" id="gmail-docs-internal-guid-b411c806-ac79-1adf-ebaf-88f505949054"><p dir="ltr" style="line-height:1.2;margin-top:0pt;margin-bottom:0pt"><br></p></b><div class="gmail_extra"><div class="gmail_extra">When I was reading documentation I found a bug in pgr_withPointsDD and I created an issue ( <a href="https://github.com/pgRouting/pgrouting/issues/979">https://github.com/pgRouting/pgrouting/issues/979</a> ) which was finally resolved but as you pointed this function still have some other bugs, So it should be our first prior to fix bugs of pgr_withPoints function.</div><div class="gmail_extra">I think it is right time to take some proposed function to official function as there are very less official functions and some experiment function to proposed function.</div><div class="gmail_extra">Seriously telling I don’t know the functionality of pgr_alphaShape but as Daniel said that there are very few non-pgRouting users, that find the AlphaShape function useful. So according to me it should be our least prior but if possible and time permit we can implement using boost in #2 road map. </div><div class="gmail_extra">Another important function is pgr_trsp. I know the functionality of this function and as you have pointed this function is not well designed. So we will fix bugs of this function ASAP as many users are using this functions. So, I’ll also go with road map #2 as it looks more stable to me. I am going to learn lot of things in release of version 3.0 of pgRouting.</div><div><br></div><div>Thanks</div><div>Aditya Pratap Singh<br><br></div><div class="gmail_quote">On Tue, Apr 3, 2018 at 9:06 PM, Vicky Vergara <<a href="mailto:vicky@georepublic.de" target="_blank">vicky@georepublic.de</a>> wrote:<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br><br> Hello fellow developers,<br><br> As you know we are rewriting pgRouting, since v2.1, to remove all "bugs",<br> like the non standard way of constructing a graph, and using more C++<br> features.<br><br> The rewrite is "almost done":<br><br> **First pair of functions**<br><br>    - pgr_alphaShape - Alpha shape computation<br>    - pgr_pointsAsPolygon - Polygon around a set of points<br><br> Both functions are tightly dependant, and I believe that they should be a<br> postGIS function as is a geometry function, and the closest one that<br> resembles this pair of functions is  ST_ConcaveHull [1] Some time ago I<br> compared both functions on [2]<br><br> I went to see how to add that function to postGIS, and for the moment that<br> is not possible, it sould be done first in GEOS, and  GEOS is so big, right<br> now I dont feel is the time to do that, but rewriting to use boost:Geometry<br> would eliminate the need of CGAL, so one less pre-requisite to build<br> pgRouting would be needed. or they can stay as they are now.<br><br> **The other function is pgr_trsp**<br><br> Which has so many problems at postgres level [4], Last year on GSoC,<br> Vidham tried to do a rewrite, confirming that the problem is much harder<br> than expected for the 3 month program. But as far as I know many people use<br> them because it has the "points" version.<br> The C++ code is not well designed [5], just a look at that, my_dijkstra<br> calls my_dijkstra and that calls my_dijkstra, it still uses pointers, so<br> great chance of memory leaks, etc.<br> Basically is **Use at your own risk**<br><br><br> Beside those functions mentioned above, now we are in the moment where<br> there are  more proposed functions [7] than official functions. and a lot<br> of deprecated functions that have to be maintained and tested because of<br> backwards compatibility.<br><br> ** So, here is my road map #1 **<br> Version 2.7 to be released on September 2018 where fix bug of<br> pgr_withPoints [8] is a must have, and the possibilities of what other<br> things it could have:<br> - New functionality done by GSoC students on "experimental section"<br> - Write a substitution (with a different name maybe pgr_dijkstraTR) for<br> pgr_trsp (one vertex to one vertex) based on dijkstra<br> - Rewrite pgr_alphaShape to use boost:graph instead of CGAL<br> - move some experimental functions up to proposed<br><br> Version 3.0 I would like it to be on September 2019<br> Where a complete cleanup of the deprecated functions would be done,<br> Move proposed functions to official pgRouting functions.<br><br> ** So, here is my road map #2 **<br><br> Version 3.0 to be released on March 2019 where fix bug of pgr_withPoints<br>[8] and Complete cleanup of the deprecated functions are a must have, and<br> the possibilities of what other things it could have:<br> - New functionality done by GSoC students on "experimental section"<br> - Write a substitution (with a different name maybe pgr_dijkstraTR) for<br>   pgr_trsp (one vertex to one vertex) based on dijkstra<br> - Rewrite pgr_alphaShape to use boost:graph instead of CGAL<br> - move some experimental functions up to proposed<br> - Move proposed functions to official pgRouting functions.<br><br> Probably this second road map will make the second version of the<br> pgRouting book a little easier to write, (please Robe, comment on this)<br><br><br> We would like comments from the community about both road maps and if<br> possible  to test proposed functions and experimental functions to help<br> decide which proposed functions can go up one level. (don't forget to open<br> issues)<br><br> Regards<br> pgRouting team<br><br><br> [1] <a href="https://postgis.net/docs/ST_ConcaveHull.html" rel="noreferrer" target="_blank">https://postgis.net/docs/ST_Co<wbr>ncaveHull.html</a><br> [2] <a href="https://github.com/cvvergara/pgrouting/issues/57" rel="noreferrer" target="_blank">https://github.com/cvvergara/p<wbr>grouting/issues/57</a><br> [3] <a href="https://github.com/pgRouting/pgrouting/blob/master/sql/" rel="noreferrer" target="_blank">https://github.com/pgRouting/p<wbr>grouting/blob/master/sql/</a><br> alpha_shape/alpha_shape.sql#L6<wbr>2<br> [4] <a href="https://github.com/pgRouting/pgrouting/tree/master/doc/trsp" rel="noreferrer" target="_blank">https://github.com/pgRouting/p<wbr>grouting/tree/master/doc/trsp</a><br> [​5] <a href="https://github.com/pgRouting/pgrouting/blob/master/include/" rel="noreferrer" target="_blank">https://github.com/pgRouting/p<wbr>grouting/blob/master/include/</a>trsp/GraphDefinition.h#L94​<br> [​6] <a href="https://github.com/pgRouting/pgrouting/tree/master/sql/pickDeliver" rel="noreferrer" target="_blank">https://github.com/pgRouting/p<wbr>grouting/tree/master/sql/pickD<wbr>eliver</a>​<br> [​7] <a href="http://docs.pgrouting.org/latest/en/proposed.html" rel="noreferrer" target="_blank">http://docs.pgrouting.org/late<wbr>st/en/proposed.html</a><br> [8] <a href="https://github.com/pgRouting/pgrouting/issues/760" rel="noreferrer" target="_blank">https://github.com/pgRouting/p<wbr>grouting/issues/760</a>​<br><br> --<br><br> Georepublic UG (haftungsbeschränkt)<br> Salzmannstraße 44,<br> 81739 München, Germany<br><br> Vicky Vergara<br> Operations Research<br><br> eMail: <a href="mailto:vicky@georepublic.de" target="_blank">vicky@georepublic.de</a><br> Web: <a href="https://georepublic.info" rel="noreferrer" target="_blank">https://georepublic.info</a><br><br> Tel: +49 (089) 4161 7698-1<br> Fax: +49 (089) 4161 7698-9<br><br> Commercial register: Amtsgericht München, HRB 181428<br> CEO: Daniel Kastl<br>
</blockquote></div><br></div></div>