Modify

Opened 13 years ago

Closed 9 years ago

Last modified 9 years ago

#7635 closed defect (fixed)

[Patch] multipolygon areas are rendered, even when they have not been completely downloaded

Reported by: cmuelle8 Owned by: team
Priority: normal Milestone:
Component: Core Version: latest
Keywords: multipolygons mapview rendering Cc: bastiK

Description

Is there a reason why incompletely downloaded MPs clutter the mapview?

There is a performance feature to turn off area rendering all together. Can we have this selectively, just for those multipolygons that have not been downloaded completely?

This would also aid correctness of MP relations - if an MP is not complete and correct, do not display it at all.

Attachments (5)

jom_incomplet_downloaded.png (27.0 KB ) - added by akks 13 years ago.
josm_incomplete.png (32.5 KB ) - added by akks 13 years ago.
way107851786.osm.bz2 (7.1 KB ) - added by simon04 10 years ago.
An example where the partial rendering is distracting and incorrect
7635.patch (3.6 KB ) - added by simon04 10 years ago.
2016-01-19-09-14-05-001.png (11.9 KB ) - added by simon04 9 years ago.

Download all attachments as: .zip

Change History (13)

comment:1 by cmuelle8, 13 years ago

I'd code it myself, if nobody steps forward. I'm asking to hear your opinion about it up front..

comment:2 by akks, 13 years ago

It can be a good feature, but it does not seem good for me to turn it on by default. The reason is very big multipolygons like forests (~ 30 ways, total 2000 nodes), that usually are not downloaded completely. If you turn off green coloriring, all wood boundaryies will look like untagged ways and confuse novices.

Here is an example of working partial rendiring:

by akks, 13 years ago

by akks, 13 years ago

Attachment: josm_incomplete.png added

in reply to:  2 comment:3 by cmuelle8, 13 years ago

Replying to akks:

If you turn off green coloriring, all wood boundaryies will look like untagged ways and confuse novices.

As I understand, coloring the way is unaffected by turning on "draw boundary of areas only". I've double checked, if I use the feature, ways themselves are rendered as expected. Partial rendering might buffle novices even more, as an edge will always exist somewhere that does not have a corresponding way in the editor.

Greetings

comment:4 by skyper, 13 years ago

+1 for making drawing of incomplete MP configurable

Maybe it would help to render them diffent.

See also #6020 and #6338.

by simon04, 10 years ago

Attachment: way107851786.osm.bz2 added

An example where the partial rendering is distracting and incorrect

by simon04, 10 years ago

Attachment: 7635.patch added

comment:5 by simon04, 10 years ago

Summary: multipolygon areas are rendered, even when they have not been completely downloaded[Patch] multipolygon areas are rendered, even when they have not been completely downloaded

attachment:7635.patch tries to detect problematic cases and stop drawing such multipolygons unless explicitly enabled (preference key mappaint.multipolygon.incomplete).

by simon04, 9 years ago

Attachment: 2016-01-19-09-14-05-001.png added

comment:6 by simon04, 9 years ago

Cc: bastiK added
Resolution: fixed
Status: newclosed

Fixed in the course of partial rendering – thanks to bastiK, see #12104

Last edited 9 years ago by Don-vip (previous) (diff)

comment:7 by cmuelle8, 9 years ago

Adding as info: In OSM any multipolygon ring, with respect to an interior, can run around clockwise or counter-clockwise around that interior.

This means for incomplete rings, we can only "guess" the side the interior will be on. Rendering an inside, partially by a buffer band or not, to a specific side of an incomplete ring will always have a chance of filling the incorrect side.

Even if we manage to correctly draw the buffer band to either the left or right side of an open ring, chance is that this side is not the side the interior is on.

If we have a complete ring, there is still the chance that role info for that complete ring is incorrect. But at least we then draw correctly with respect to the role information provided. To determine if role info is plausible, the whole MP data is needed.

comment:8 by Don-vip, 9 years ago

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.