Hi Steve,<div><br></div><div>It's a good idea.</div><div>Though I assume we won't get overloaded with submissions ;-)</div><div><br></div><div>I think we could add your submission guidelines as a README in the repository root: <a href="https://github.com/pgRouting/pgrouting-contrib">https://github.com/pgRouting/pgrouting-contrib</a></div>
<div>It's content then shows up automatically beneath the directories.</div><div><br></div><div>I guess the amount of files will vary depending on the type of document. Some contributions may not need sample data (for example the template script), some might better have an own directory. So I would make it more pragmatic and see the guidelines as a recommendation. Though the result should be usable and understandable and it should be documented, who wrote it and under which license ... so not like the files I added ;-)</div>
<div><br></div><div>Daniel</div><div><br></div><div><br><div class="gmail_quote">On Mon, Dec 5, 2011 at 10:40 AM, Stephen Woodbridge <span dir="ltr"><<a href="mailto:woodbri@swoodbridge.com">woodbri@swoodbridge.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Daniel,<br>
<br>
Ok, this makes some sense. I would like us to try and set some minimum requirements for the contrib submissions. Like when a file(s) is contributed it gets a named like:<br>
<whatever>-cod.sql - example code<br>
<whatever>-doc.txt - file with some minimum doc in it<br>
<whatever>-dat.sql - sample data in pg_dump -fP format compressed with gzip<br>
<br>
<br>
For example of <whatever>-doc.txt:<br>
<br>
Author<br>
- name<br>
- email<br>
- date<br>
- pgRouting version<br>
- copyright<br>
Overview<br>
- what does this file do or show from an example point of view<br>
Assumptions<br>
- ....<br>
How do I use it<br>
- how to load and prep <whatever>-dat.sql.gz<br>
- list of sql statements to run the example code and the expected results.<br>
<br>
We can relax these on a case by case basis after the PSC has had a chance to review potential submissions. This is less to raise a barrier and more to set some quality standards for pgRouting. People can always post code in blogs and private git repositories, etc without meeting these standards. Also if pgRouting changes in some way and the examples not longer work and no one is interested in updating them, then they should be removed.<br>
<br>
My 2 cents,<br>
-Steve<div class="im"><br>
<br>
On 12/4/2011 8:13 PM, Daniel Kastl wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
Hi Steve,<br>
<br>
I fully agree. Well, the idea of the pgrouting-contrib repository is to<br>
collect stuff like that: wrapper examples, scripts like the one to<br>
create a pgRouting database template, etc..<br>
The purpose of this repository is not to provide widely tested and<br>
perfect wrappers. But as Worth said, it can be of great help to have<br>
some examples if you want to write your first own wrapper function. And<br>
it's not a good idea to put too many wrapper functions into core<br>
library. There are already too many.<br>
<br>
So I think it's better to publish "unfinished" and custom code samples,<br>
because that still helps users more than no examples at all, right? ;-)<br>
In the worst case it's just there and can be taken as a reference to<br>
answer questions in the mailing list, in the best case someone<br>
contributes feedback or even improvements.<br>
<br>
Unfortunately it pgrouting-contrib has never grown beyond the one<br>
wrapper and a few other things. Probably it wasn't explained clear<br>
enough that everyone is welcome to add their examples there. Or it's too<br>
difficult to contribute.<br>
<br>
Daniel<br>
<br>
<br>
<br>
<br>
On Mon, Dec 5, 2011 at 9:39 AM, Stephen Woodbridge<br></div><div><div class="h5">
<<a href="mailto:woodbri@swoodbridge.com" target="_blank">woodbri@swoodbridge.com</a> <mailto:<a href="mailto:woodbri@swoodbridge.com" target="_blank">woodbri@swoodbridge.<u></u>com</a>>> wrote:<br>
<br>
Hi Worth and all,<br>
<br>
Thanks for the offer but no need for that now.<br>
My main concern was that looking at it, it does not seem that we<br>
want to make it a part of the core functionality for a few reasons:<br>
<br>
o it is not documented<br>
o as pointed out it is not generalized and has some assumptions<br>
o a lot of the code could be simplified and made more efficient nor<br>
is it the best example, although I'm sure it did the job that was<br>
needed at the time<br>
o our experience has been that wrappers that get release as examples<br>
tend to get used as the "main" API to pgrouting and therefore need<br>
to be clear, easy to follow and well documented.<br>
<br>
Having said that, we need some nice wrappers. I have written my own<br>
wrappers, but alas they also suffer from many of the same ills, I<br>
mentioned because I wrote them for my own use/reuse and not for the<br>
general public. If I can find some time it is on my list of things<br>
to do for pgrouting.<br>
<br>
Thanks,<br>
-Steve<br>
<br>
<br>
On 12/4/2011 6:35 PM, Worth Lutz wrote:<br>
<br>
As Daniel noted, the function will not work with all types of data.<br>
<br>
I cannot remember now but I had to change something in it to make it<br>
work with my data. I could diff against the original and tell<br>
you what<br>
if anyone is interested.<br>
<br>
It is a great example though. It taught me how to use pl/pgsql<br>
to solve<br>
problems I was having.<br>
<br>
Worth Lutz<br>
<br>
*From: * pgrouting-users-bounces@lists. <a href="http://osgeo.org" target="_blank">osgeo.org</a><br></div></div>
<mailto:<a href="mailto:pgrouting-users-bounces@lists.osgeo.org" target="_blank">pgrouting-users-<u></u>bounces@lists.osgeo.org</a>><br>
[mailto:<a href="mailto:pgrouting-users-" target="_blank">pgrouting-users-</a> <a href="mailto:bounces@lists.osgeo.org" target="_blank">bounces@lists.osgeo.org</a><div class="im"><br>
<mailto:<a href="mailto:pgrouting-users-bounces@lists.osgeo.org" target="_blank">pgrouting-users-<u></u>bounces@lists.osgeo.org</a>>] *On Behalf Of<br>
*Daniel Kastl<br>
*Sent:* Sunday, December 04, 2011 10:39 AM<br>
*To:* pgRouting users mailing list<br>
*Subject:* Re: [pgrouting-users] routing_core_smart problem<br>
<br>
<br>
On Mon, Dec 5, 2011 at 12:28 AM, Stephen Woodbridge<br>
<<a href="mailto:woodbri@swoodbridge.com" target="_blank">woodbri@swoodbridge.com</a> <mailto:<a href="mailto:woodbri@swoodbridge.com" target="_blank">woodbri@swoodbridge.<u></u>com</a>><br></div>
<mailto:<a href="mailto:woodbri@swoodbridge" target="_blank">woodbri@swoodbridge</a>. com<div><div class="h5"><br>
<mailto:<a href="mailto:woodbri@swoodbridge.com" target="_blank">woodbri@swoodbridge.<u></u>com</a>>>> wrote:<br>
<br>
Hi Daniel,<br>
<br>
Who wrote the routing_core_smart.sql functions?<br>
<br>
Anton<br>
<br>
Is there any documentation of them? It would be nice if we had:<br>
<br>
Right ... would be nice to have ;-)<br>
<br>
I actually just decided to publish the wrapper, because there were<br>
several questions in the past how the pgRouting demo could start the<br>
route from/to the point you clicked.<br>
<br>
1. description of the major public entry points and what the<br>
arguments should be.<br>
<br>
It even needs an extra table "network_statistics" to work.<br>
<br>
But on the other hand it was supposed to be an example of<br>
writing a more<br>
complex wrapper function. It's not really a generic function and<br>
may not<br>
work with any type of data.<br>
<br>
There are probably several possibilities to improve the functions.<br>
<br>
Daniel<br>
<br>
2. comments in the code that explain what the overview<br>
strategy is<br>
and/or a high level pseudocode of how it executes<br>
<br>
Thanks,<br>
-Steve<br>
<br>
<br>
<br>
On 12/2/2011 9:57 PM, Daniel Kastl wrote:<br>
<br>
Hi Lutz,<br>
<br>
Thank you very much for sharing this information!<br>
I have also created this ticket:<br>
<a href="https://github.com/pgRouting/" target="_blank">https://github.com/pgRouting/</a> pgrouting-contrib/issues/3<br>
<<a href="https://github.com/pgRouting/pgrouting-contrib/issues/3" target="_blank">https://github.com/pgRouting/<u></u>pgrouting-contrib/issues/3</a>><br>
<br>
So you're talking about the following two lines?<br>
<br>
* Line 191<br>
<a href="https://github.com/pgRouting/" target="_blank">https://github.com/pgRouting/</a> pgrouting-contrib/blob/master/<br>
wrapper/routing_core_smart. sql#L191<br>
<<a href="https://github.com/pgRouting/pgrouting-contrib/blob/master/wrapper/routing_core_smart.sql#L191" target="_blank">https://github.com/pgRouting/<u></u>pgrouting-contrib/blob/master/<u></u>wrapper/routing_core_smart.<u></u>sql#L191</a>><br>
* Line 205<br>
<br>
<br>
<a href="https://github.com/pgRouting/" target="_blank">https://github.com/pgRouting/</a> pgrouting-contrib/blob/master/<br>
wrapper/routing_core_smart. sql#L205<br>
<<a href="https://github.com/pgRouting/pgrouting-contrib/blob/master/wrapper/routing_core_smart.sql#L205" target="_blank">https://github.com/pgRouting/<u></u>pgrouting-contrib/blob/master/<u></u>wrapper/routing_core_smart.<u></u>sql#L205</a>><br>
<br>
Best regards,<br>
Daniel<br>
<br>
<br>
On Sat, Dec 3, 2011 at 4:26 AM, Worth Lutz<br>
<<a href="mailto:wal3@mindspring.com" target="_blank">wal3@mindspring.com</a> <mailto:<a href="mailto:wal3@mindspring.com" target="_blank">wal3@mindspring.com</a>><br>
<mailto:<a href="mailto:wal3@mindspring.com" target="_blank">wal3@mindspring.com</a> <mailto:<a href="mailto:wal3@mindspring.com" target="_blank">wal3@mindspring.com</a>>><br>
<br>
<mailto:<a href="mailto:wal3@mindspring.com" target="_blank">wal3@mindspring.com</a> <mailto:<a href="mailto:wal3@mindspring.com" target="_blank">wal3@mindspring.com</a>><br>
<mailto:<a href="mailto:wal3@mindspring.com" target="_blank">wal3@mindspring.com</a> <mailto:<a href="mailto:wal3@mindspring.com" target="_blank">wal3@mindspring.com</a>>>><u></u>> wrote:<br>
<br>
Hi all,<br>
<br>
I have found what appears to be a problem in one of the<br>
routing_core_smart functions.<br>
<br>
In the function locate_point_as_geometry there are two<br>
lines which<br>
call the function geometryfromtext.<br>
<br>
These functions are called when line_substring returns a<br>
point.<br>
<br>
In the geometryfromtext function calls, there is no<br>
second argument<br>
for the srid. This produces a geometry with a “-1” for<br>
the srid.<br>
<br>
When I use shootingstar_smart in my script, MapServer<br>
fails on<br>
trying to work with two geometries with different srids.<br>
<br>
Adding the second argument to the geometryfromtext<br>
function fixed<br>
this problem. The only geometry with the “-1” srid was<br>
the one<br>
created by these function calls.<br>
<br>
I hope this helps someone.<br>
<br>
Worth Lutz<br>
<br>
<br>
______________________________ _________________<br>
Pgrouting-users mailing list<br>
<br>
Pgrouting-users@lists.osgeo. org<br>
<mailto:<a href="mailto:Pgrouting-users@lists.osgeo.org" target="_blank">Pgrouting-users@lists.<u></u>osgeo.org</a>><br>
<mailto:<a href="mailto:Pgrouting-users@lists" target="_blank">Pgrouting-users@lists</a>. <a href="http://osgeo.org" target="_blank">osgeo.org</a><br>
<mailto:<a href="mailto:Pgrouting-users@lists.osgeo.org" target="_blank">Pgrouting-users@lists.<u></u>osgeo.org</a>>><br></div></div>
<mailto:<a href="mailto:Pgrouting-users@lists" target="_blank">Pgrouting-users@lists</a>. <a href="http://osgeo.org" target="_blank">osgeo.org</a><br>
<mailto:<a href="mailto:Pgrouting-users@lists.osgeo.org" target="_blank">Pgrouting-users@lists.<u></u>osgeo.org</a>><br>
<br>
<mailto:<a href="mailto:Pgrouting-users@lists" target="_blank">Pgrouting-users@lists</a>. <a href="http://osgeo.org" target="_blank">osgeo.org</a><br>
<mailto:<a href="mailto:Pgrouting-users@lists.osgeo.org" target="_blank">Pgrouting-users@lists.<u></u>osgeo.org</a>>>><br>
<br>
<br>
<a href="http://lists.osgeo.org/" target="_blank">http://lists.osgeo.org/</a> mailman/listinfo/pgrouting- users<div class="im"><br>
<<a href="http://lists.osgeo.org/mailman/listinfo/pgrouting-users" target="_blank">http://lists.osgeo.org/<u></u>mailman/listinfo/pgrouting-<u></u>users</a>><br>
<br>
<br>
<br>
<br>
--<br>
Georepublic UG & Georepublic Japan<br>
<br>
eMail: <a href="mailto:daniel.kastl@georepublic.de" target="_blank">daniel.kastl@georepublic.de</a><br>
<mailto:<a href="mailto:daniel.kastl@georepublic.de" target="_blank">daniel.kastl@<u></u>georepublic.de</a>><br>
<mailto:<a href="mailto:daniel.kastl@" target="_blank">daniel.kastl@</a> <a href="http://georepublic.de" target="_blank">georepublic.de</a><br>
<mailto:<a href="mailto:daniel.kastl@georepublic.de" target="_blank">daniel.kastl@<u></u>georepublic.de</a>>><br></div>
<mailto:<a href="mailto:daniel.kastl@" target="_blank">daniel.kastl@</a> <a href="http://georepublic.de" target="_blank">georepublic.de</a><br>
<mailto:<a href="mailto:daniel.kastl@georepublic.de" target="_blank">daniel.kastl@<u></u>georepublic.de</a>><br>
<br>
<mailto:<a href="mailto:daniel.kastl@" target="_blank">daniel.kastl@</a> <a href="http://georepublic.de" target="_blank">georepublic.de</a><div class="im"><br>
<mailto:<a href="mailto:daniel.kastl@georepublic.de" target="_blank">daniel.kastl@<u></u>georepublic.de</a>>>><br>
Web: <a href="http://georepublic.de" target="_blank">http://georepublic.de</a> <<a href="http://georepublic.de/" target="_blank">http://georepublic.de/</a>><br>
<br>
<br>
<br>
<br>
______________________________ _________________<br>
Pgrouting-users mailing list<br>
Pgrouting-users@lists.osgeo. org<br>
<mailto:<a href="mailto:Pgrouting-users@lists.osgeo.org" target="_blank">Pgrouting-users@lists.<u></u>osgeo.org</a>><br></div>
<mailto:<a href="mailto:Pgrouting-users@lists" target="_blank">Pgrouting-users@lists</a>. <a href="http://osgeo.org" target="_blank">osgeo.org</a><br>
<mailto:<a href="mailto:Pgrouting-users@lists.osgeo.org" target="_blank">Pgrouting-users@lists.<u></u>osgeo.org</a>>><br>
<br>
<a href="http://lists.osgeo.org/" target="_blank">http://lists.osgeo.org/</a> mailman/listinfo/pgrouting- users<br>
<<a href="http://lists.osgeo.org/mailman/listinfo/pgrouting-users" target="_blank">http://lists.osgeo.org/<u></u>mailman/listinfo/pgrouting-<u></u>users</a>><br>
<br>
<br>
______________________________ _________________<br>
Pgrouting-users mailing list<br>
Pgrouting-users@lists.osgeo. org<br>
<mailto:<a href="mailto:Pgrouting-users@lists.osgeo.org" target="_blank">Pgrouting-users@lists.<u></u>osgeo.org</a>><br>
<mailto:<a href="mailto:Pgrouting-users@lists" target="_blank">Pgrouting-users@lists</a>. <a href="http://osgeo.org" target="_blank">osgeo.org</a><br>
<mailto:<a href="mailto:Pgrouting-users@lists.osgeo.org" target="_blank">Pgrouting-users@lists.<u></u>osgeo.org</a>>><br>
<a href="http://lists.osgeo.org/" target="_blank">http://lists.osgeo.org/</a> mailman/listinfo/pgrouting- users<div class="im"><br>
<<a href="http://lists.osgeo.org/mailman/listinfo/pgrouting-users" target="_blank">http://lists.osgeo.org/<u></u>mailman/listinfo/pgrouting-<u></u>users</a>><br>
<br>
<br>
<br>
--<br>
Georepublic UG & Georepublic Japan<br>
eMail: <a href="mailto:daniel.kastl@georepublic.de" target="_blank">daniel.kastl@georepublic.de</a><br>
<mailto:<a href="mailto:daniel.kastl@georepublic.de" target="_blank">daniel.kastl@<u></u>georepublic.de</a>> <mailto:<a href="mailto:daniel.kastl@" target="_blank">daniel.kastl@</a><br></div><div class="im">
<a href="http://georepublic.de" target="_blank">georepublic.de</a> <mailto:<a href="mailto:daniel.kastl@georepublic.de" target="_blank">daniel.kastl@<u></u>georepublic.de</a>>><br>
Web: <a href="http://georepublic.de" target="_blank">http://georepublic.de</a> <<a href="http://georepublic.de/" target="_blank">http://georepublic.de/</a>><br>
<br>
No virus found in this message.<br>
Checked by AVG - <a href="http://www.avg.com" target="_blank">www.avg.com</a> <<a href="http://www.avg.com" target="_blank">http://www.avg.com</a>><br>
<<a href="http://www.avg.com" target="_blank">http://www.avg.com</a>><br>
<br>
Version: 2012.0.1873 / Virus Database: 2102/4656 - Release Date:<br>
12/04/11<br>
<br>
<br>
<br>
______________________________ _________________<br>
Pgrouting-users mailing list<br>
Pgrouting-users@lists.osgeo. org<br></div>
<mailto:<a href="mailto:Pgrouting-users@lists.osgeo.org" target="_blank">Pgrouting-users@lists.<u></u>osgeo.org</a>><br>
<a href="http://lists.osgeo.org/" target="_blank">http://lists.osgeo.org/</a> mailman/listinfo/pgrouting- users<br>
<<a href="http://lists.osgeo.org/mailman/listinfo/pgrouting-users" target="_blank">http://lists.osgeo.org/<u></u>mailman/listinfo/pgrouting-<u></u>users</a>><br>
<br>
<br>
______________________________ _________________<br>
Pgrouting-users mailing list<br>
Pgrouting-users@lists.osgeo. org<br>
<mailto:<a href="mailto:Pgrouting-users@lists.osgeo.org" target="_blank">Pgrouting-users@lists.<u></u>osgeo.org</a>><br>
<a href="http://lists.osgeo.org/" target="_blank">http://lists.osgeo.org/</a> mailman/listinfo/pgrouting- users<div class="im"><br>
<<a href="http://lists.osgeo.org/mailman/listinfo/pgrouting-users" target="_blank">http://lists.osgeo.org/<u></u>mailman/listinfo/pgrouting-<u></u>users</a>><br>
<br>
<br>
<br>
<br>
--<br>
Georepublic UG & Georepublic Japan<br>
eMail: <a href="mailto:daniel.kastl@georepublic.de" target="_blank">daniel.kastl@georepublic.de</a> <mailto:<a href="mailto:daniel.kastl@georepublic.de" target="_blank">daniel.kastl@<u></u>georepublic.de</a>><br>
Web: <a href="http://georepublic.de" target="_blank">http://georepublic.de</a> <<a href="http://georepublic.de/" target="_blank">http://georepublic.de/</a>><br>
<br>
<br></div><div class="im">
______________________________<u></u>_________________<br>
Pgrouting-users mailing list<br>
<a href="mailto:Pgrouting-users@lists.osgeo.org" target="_blank">Pgrouting-users@lists.osgeo.<u></u>org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/pgrouting-users" target="_blank">http://lists.osgeo.org/<u></u>mailman/listinfo/pgrouting-<u></u>users</a><br>
</div></blockquote><div class="HOEnZb"><div class="h5">
<br>
______________________________<u></u>_________________<br>
Pgrouting-users mailing list<br>
<a href="mailto:Pgrouting-users@lists.osgeo.org" target="_blank">Pgrouting-users@lists.osgeo.<u></u>org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/pgrouting-users" target="_blank">http://lists.osgeo.org/<u></u>mailman/listinfo/pgrouting-<u></u>users</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><span style="font-family:arial, sans-serif;font-size:13px;border-collapse:collapse">Georepublic UG & Georepublic Japan<br>eMail: <a href="mailto:daniel.kastl@georepublic.de" style="color:rgb(66, 99, 171)" target="_blank">daniel.kastl@georepublic.de</a><br>
Web: <a href="http://georepublic.de/" style="color:rgb(66, 99, 171)" target="_blank">http://georepublic.de</a></span><br>
</div>