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

Prevent double way and node additions #1017

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

Prevent double way and node additions #1017

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

Comments

@openstreetmap-trac
Copy link

Reporter: stoecker
[Submitted to the original trac issue database at 2.22pm, Tuesday, 8th July 2008]

In the last time the OSM server and potlatch a very slow. This results into many duplicate ways and nodes in the database.

Please fix this:
a) Make clearly visible, that not all data is loaded yet, so a user does not draw a way when data is incomplete.
b) Prevent upload/download bugs

This is a major topic, as the number of double stuff increased a lot in the last month.

@openstreetmap-trac
Copy link
Author

Author: richard
[Added to the original trac issue at 11.02am, Wednesday, 9th July 2008]

Server response times should be significantly speeded up later this week with Potlatch 0.10 and a major rewrite of amf_controller.rb. I'm working on improvements to make the "loading ways" display more obvious.

Of course, everyone can help improve speed by donating new servers etc. to OSM. :)

@openstreetmap-trac
Copy link
Author

Author: schuetzm[at]gmx.net
[Added to the original trac issue at 12.57pm, Wednesday, 9th July 2008]

Why can't editing simply be disabled until all ways are loaded?

@openstreetmap-trac
Copy link
Author

Author: richard
[Added to the original trac issue at 1.04pm, Wednesday, 9th July 2008]

Because "all ways are loaded" is not that simple.

On the initial load, maybe it's possible. But when you're panning around the map, you still might be editing in an area for which all the data has been loaded: the "blank" bit is off to the side in the newly panned bit. And if you only disable editing for the initial load, the user would think "oh, I can edit the blank bit, because it hasn't been disabled".

So, like I say, the best answer appears to be to make the "loading ways" display more obvious.

@openstreetmap-trac
Copy link
Author

Author: stoecker
[Added to the original trac issue at 1.10pm, Wednesday, 9th July 2008]

Probably copy from merkaartor.

Where josm uses a yellow line to indicate download area, merkaartor uses an are fill with small red dots to mark the "keep hand away" area. I think that should work for potlatch also.

Make all area red-dotted until loading is finished. The remove the red dots for the finished areas.

@openstreetmap-trac
Copy link
Author

Author: richard
[Added to the original trac issue at 1.27pm, Wednesday, 9th July 2008]

Not a bad idea, but would be very complicated to implement. Every time you pan, Potlatch loads from a new bbox, so you would typically have some 20 overlapping bboxes. Calculating the boundary line of this area is not trivial.

It might be good as a long-term aspiration but, with 8bn other demands on time at the moment, I think the more prominent "loading ways" display, and the faster server response times, are more likely to get results faster.

@openstreetmap-trac
Copy link
Author

Author: stoecker
[Added to the original trac issue at 1.47pm, Wednesday, 9th July 2008]

Hmm. I wouldn't calculate the boundary at all.

When loading state is finished or display is moved/zoomed, fill the current display with dots and now remove all dots inside loaded areas. Very likely that can be done in background and clipped into display when complete?

@openstreetmap-trac
Copy link
Author

Author: richard
[Added to the original trac issue at 1.49pm, Wednesday, 9th July 2008]

Very good idea.

Ok, will give it some thought for 1.0.

@openstreetmap-trac
Copy link
Author

Author: schuetzm[at]gmx.net
[Added to the original trac issue at 1.54pm, Wednesday, 9th July 2008]

Replying to [comment:3 richard]:

Because "all ways are loaded" is not that simple.

On the initial load, maybe it's possible. But when you're panning around the map, you still might be editing in an area for which all the data has been loaded: the "blank" bit is off to the side in the newly panned bit. And if you only disable editing for the initial load, the user would think "oh, I can edit the blank bit, because it hasn't been disabled".

So, like I say, the best answer appears to be to make the "loading ways" display more obvious.

I can't follow you here... If you can display "loading ways" whenever ways are loaded, you surely can also disable editing while "loading ways" is displayed, right?

@openstreetmap-trac
Copy link
Author

Author: richard
[Added to the original trac issue at 2.00pm, Wednesday, 9th July 2008]

Oh, sure, you can. But the point is that a user might want to continue editing a loaded area, while another area loads - I do this all the time. stoecker's suggestion sounds like a good way of making it clear to the novice, without restricting the knowledgeable user's options.

@openstreetmap-trac
Copy link
Author

Author: Richard
[Added to the original trac issue at 6.58pm, Wednesday, 9th July 2008]

Note to self - comp.graphics.algorithms 2.04 (http://www.faqs.org/faqs/graphics/algorithms-faq/) is relevant - basically build a list of all the latitudes of already loaded bboxes, then use those to work out the rectangles within the current bbox.

@openstreetmap-trac
Copy link
Author

Author: Richard
[Added to the original trac issue at 6.55am, Thursday, 10th July 2008]

0.10 (awaiting deployment) contains a much more visible, and localised, "loading ways" display. With that in mind I think any further changes can be changed from a defect to an enhancement.

@openstreetmap-trac
Copy link
Author

Author: iandees
[Added to the original trac issue at 7.37pm, Monday, 9th September 2013]

Cleaning aging tickets.

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