Skip to content
This repository has been archived by the owner on Jul 24, 2021. It is now read-only.

Potlatch 2: join nodes function creates duplicate nodes in ways #4375

Closed
openstreetmap-trac opened this issue Jul 23, 2021 · 6 comments
Closed

Comments

@openstreetmap-trac
Copy link

Reporter: Firefly!
[Submitted to the original trac issue database at 1.36pm, Monday, 23rd April 2012]

It seems to be a bug inside the Potlatch function "join nodes" (key J). When I stick together two (or more) areas by joining nodes, sometimes the OSM inspector tool tells me, I have created duplicate nodes in one of the resulting ways.
f.e.
http://www.openstreetmap.org/browse/node/1728680163
http://www.openstreetmap.org/browse/node/1706551424

I can reproduce it as follow:
Way 1 contains node 1a and node 1b. Way 2 contains node 2.
I join node 2 with node 1a. Later I join the resulting node with 1b.
After uploading I will get the duplicate node in way.

Normally this situation will not happen so often, but at edges of three or more areas you can have a lot of nodes and it is almost impossible to take care for all ...

Best regards

@openstreetmap-trac
Copy link
Author

Author: Oli-Wan
[Added to the original trac issue at 11.29am, Wednesday, 25th April 2012]

This is actually a duplicate of #2501; the problem has been known for a long time. However, I can confirm that the procedure outlined above actually "works" and reproduces the error.

@openstreetmap-trac
Copy link
Author

Author: stevage
[Added to the original trac issue at 4.55pm, Saturday, 5th May 2012]

I had trouble reliably reproducing this with these instructions. But I've fixed #2501, so it's pretty likely this is fixed too. Perhaps when this fix is in production, someone could verify and close.

systemed/potlatch2#65

@openstreetmap-trac
Copy link
Author

Author: Firefly!
[Added to the original trac issue at 1.17pm, Wednesday, 23rd May 2012]

Unfortunately it is still possible to create duplicate nodes with the decribed way, but I think the problem is more simple and follows a basically bug:

If you draw a closed way (f.e. for a landuse) you can see in the list of nodes mostly the first and the last node with the same ID.

f.e. http://www.openstreetmap.org/browse/way/164694686

(With my basic knowledge I can find in your bugfix 2501-source code sample only a check for two identical nodes after each other, but not for the case that first and last node are the same. And it seems that more than two identical nodes will also handled maybe incomplete...)

Best regards

@openstreetmap-trac
Copy link
Author

Author: stevage
[Added to the original trac issue at 11.21pm, Wednesday, 23rd May 2012]

I'm not quite following - the first and last nodes of a closed way are supposed to be the same. That's how closed ways work. So ABCDA is ok. ABACDA is marginal but ok. ABACA is likewise marginal but ok. ABABA - no idea. AABCD is always wrong.

Do you have any instructions for producing an AABCD sequence with the patch?

@openstreetmap-trac
Copy link
Author

Author: Firefly!
[Added to the original trac issue at 9.19am, Thursday, 24th May 2012]

Sorry, it is as you said. I had to find out at a lot of other closed ways that the sequence ABCDA is correct.
This seems to be more a problem of the visualisation at http://www.openstreetmap.org/browse/way/* and specially for the start node A at http://www.openstreetmap.org/browse/node/* (f.e. "Part of: 123456789, 123456789") ...

About creation of other incorrect sequences like AABCD: I tried a lot variations now, but I could not reproduce it anymore. I will continue to observe it sometimes with OSMI, because in the last two months I did produce a lot of such duplicates. ...
By the way, also with Join-nodes it is possible to create a single node way -> see ticket 4378.

Thanks for your support and best regards!

@openstreetmap-trac
Copy link
Author

Author: Richard
[Added to the original trac issue at 3.47pm, Tuesday, 11th September 2012]

This appears to be fixed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant