[prev in list] [next in list] [prev in thread] [next in thread] 

List:       postgis-users
Subject:    Re: [postgis-users] Need a method for "noding" a street network
From:       Nicolas Ribot <nicolas.ribot () gmail ! com>
Date:       2013-05-10 10:45:53
Message-ID: CAGAwT=2PJV9chUQ4NQGZnGtNHnmq1b40XOb5vjMBv-k31Qv43A () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


On 10 May 2013 10:34, Sandro Santilli <strk@keybit.net> wrote:

> On Thu, May 09, 2013 at 02:53:29PM -0400, Stephen Woodbridge wrote:
> > On 5/9/2013 1:27 PM, Nicolas Ribot wrote:
> > >Steve,
> > >
> > >I modified your script to include the final step gathering all needed
> > >input lines in the results table.
> > >I also corrected the linestring ends insertion queries:
> > >
> > >...quote_ident(n_geom) || ') <> ''LINESTRING'' '; to become:
> > >...quote_ident(n_geom) || ') = ''LINESTRING'' ';
> > >
> > >which inserted null values for geom column as st_startpoint(geom) is
> > >null for any geometry <> LINESTRING.
> >
> > Duh, my bad on that and you even mentioned it. I'm pleading a nasty
> > sinus cold which has scramble my brains.
> >
> > I would like to include this in pgRouting, if that is OK with you. I
> > would also like to credit you for your effort to develop this tool,
> > so if you would like to add a comment and copyright that would be
> > great.
> >
> > >It seems to work well on bdaways dataset and costa rica highways.
> > >
> > >I will add some snapping operations as suggested by Sandro and try to
> > >see if all segments are noded.
> > >(Topology still creating after 4270 ms)
> >
> > The snapping would be nice. I'm wondering if Sandro could possible
> > use a variant of this to speed up the topology creation?
>
> Possibly. But topology building also has to setup explicit edge linking
> (which edge comes next by walking on each side of the edge) and handle
> face creation.
>
> I've actually a big refactor in mind for topology building, in that the
> current model of snapping new-comers to pre-existing primitives has a
> conceptual bug (there's no way for existing edges to be immutable).
>
> But anyway ST_CreateTopoGeo would have all input at once so shouldn't
> suffer from that bug. It'd be great if Nicolas wanted to take a look :)
> IIRC that function wasn't optimized as I tought it wasn't as useful as
> the ones building the topology incrementally... I'm pretty sure there's
> performance to squeeze out of it.
>

I Sandro,

I canceled the topology creation on costa rica dataset after 1 hour of run.
I will try with Steve dataset, which is smaller.

Nicolas

>
> --strk;
>

[Attachment #5 (text/html)]

<div dir="ltr">On 10 May 2013 10:34, Sandro Santilli <span dir="ltr">&lt;<a \
href="mailto:strk@keybit.net" target="_blank">strk@keybit.net</a>&gt;</span> \
wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex">

<div class="im">On Thu, May 09, 2013 at 02:53:29PM -0400, Stephen Woodbridge \
wrote:<br> &gt; On 5/9/2013 1:27 PM, Nicolas Ribot wrote:<br>
&gt; &gt;Steve,<br>
&gt; &gt;<br>
&gt; &gt;I modified your script to include the final step gathering all needed<br>
&gt; &gt;input lines in the results table.<br>
&gt; &gt;I also corrected the linestring ends insertion queries:<br>
&gt; &gt;<br>
&gt; &gt;...quote_ident(n_geom) || &#39;) &lt;&gt; &#39;&#39;LINESTRING&#39;&#39; \
&#39;; to become:<br> &gt; &gt;...quote_ident(n_geom) || &#39;) = \
&#39;&#39;LINESTRING&#39;&#39; &#39;;<br> &gt; &gt;<br>
&gt; &gt;which inserted null values for geom column as st_startpoint(geom) is<br>
&gt; &gt;null for any geometry &lt;&gt; LINESTRING.<br>
&gt;<br>
&gt; Duh, my bad on that and you even mentioned it. I&#39;m pleading a nasty<br>
&gt; sinus cold which has scramble my brains.<br>
&gt;<br>
&gt; I would like to include this in pgRouting, if that is OK with you. I<br>
&gt; would also like to credit you for your effort to develop this tool,<br>
&gt; so if you would like to add a comment and copyright that would be<br>
&gt; great.<br>
&gt;<br>
&gt; &gt;It seems to work well on bdaways dataset and costa rica highways.<br>
&gt; &gt;<br>
&gt; &gt;I will add some snapping operations as suggested by Sandro and try to<br>
&gt; &gt;see if all segments are noded.<br>
&gt; &gt;(Topology still creating after 4270 ms)<br>
&gt;<br>
&gt; The snapping would be nice. I&#39;m wondering if Sandro could possible<br>
&gt; use a variant of this to speed up the topology creation?<br>
<br>
</div>Possibly. But topology building also has to setup explicit edge linking<br>
(which edge comes next by walking on each side of the edge) and handle<br>
face creation.<br>
<br>
I&#39;ve actually a big refactor in mind for topology building, in that the<br>
current model of snapping new-comers to pre-existing primitives has a<br>
conceptual bug (there&#39;s no way for existing edges to be immutable).<br>
<br>
But anyway ST_CreateTopoGeo would have all input at once so shouldn&#39;t<br>
suffer from that bug. It&#39;d be great if Nicolas wanted to take a look :)<br>
IIRC that function wasn&#39;t optimized as I tought it wasn&#39;t as useful as<br>
the ones building the topology incrementally... I&#39;m pretty sure there&#39;s<br>
performance to squeeze out of it.<br></blockquote><div><br></div><div style>I \
Sandro,</div><div style><br></div><div style>I canceled the topology creation on \
costa rica dataset after 1 hour of run.</div><div style>I will try with Steve \
dataset, which is smaller.</div>

<div style><br></div><div style>Nicolas </div><blockquote class="gmail_quote" \
style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> <br>
--strk;<br>
</blockquote></div><br></div></div>



_______________________________________________
postgis-users mailing list
postgis-users@lists.osgeo.org
http://lists.osgeo.org/cgi-bin/mailman/listinfo/postgis-users


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic