Modify

Opened 5 years ago

Closed 5 years ago

#10253 closed defect (fixed)

don't create an associatedStreet-relation as default

Reported by: brogo Owned by: team
Priority: normal Milestone:
Component: Plugin terracer Version:
Keywords: Cc:

Description

Please set the option to create a associatedStreet-relation by default as "disabled".

Many associatedStreet-relation are created accidentially and doesn't contain the needed information (street-member). If someone relly works with aS-relations he can enable this option.

Attachments (0)

Change History (6)

comment:1 Changed 5 years ago by stephan75

Indeed we need a change of that feature!

According to the thread in German subforum http://forum.openstreetmap.org/viewtopic.php?id=26238 there is an overpass-turbo query how you can display occurence of that wrong relations:

/* choose here any area for that query that can be found by Nominatim */
{{nominatimArea:Bayern}}
(._; )->.area;

/* find all relations of type=associatedStreet */
rel[type=associatedStreet](area.area)->.allASRelations;

/* now find all relations of type=associatedStreet, that have a way as member with role "street" */
way(r.allASRelations:"street");rel(bw:"street")[type=associatedStreet]->.relationsWithRoleStreet;

/* Calculate the difference of these two data pools */
(.allASRelations; - .relationsWithRoleStreet;);

/* choose ways and nodes of that result and display it */
(._; >>;);
out meta;}}}

... so paste this into http://overpass-turbo.eu and execute that query.

comment:2 Changed 5 years ago by stephan75

Indeed we need a change of that feature!

According to the thread in German subforum http://forum.openstreetmap.org/viewtopic.php?id=26238 there is an overpass-turbo query how you can display occurence of that wrong relations:

/* choose here any area for that query that can be found by Nominatim */
{{nominatimArea:Bayern}}
(._; )->.area;

/* find all relations of type=associatedStreet */
rel[type=associatedStreet](area.area)->.allASRelations;

/* now find all relations of type=associatedStreet, that have a way as member with role "street" */
way(r.allASRelations:"street");rel(bw:"street")[type=associatedStreet]->.relationsWithRoleStreet;

/* Calculate the difference of these two data pools */
(.allASRelations; - .relationsWithRoleStreet;);

/* choose ways and nodes of that result and display it */
(._; >>;);
out meta;

... so paste this into http://overpass-turbo.eu and execute that query.

comment:3 Changed 5 years ago by Klumbumbus

Please fix this. This bug created thousands of useless and broken associatesStreet relations. (At the moment there are 1791 of these broken relations just in Bayern!)
Please atleast set the option to create a associatedStreet relation by default to disabled.
In my opinion it would even be better to completely remove the creation of associatedStreet relations with terracer plugin.

comment:4 Changed 5 years ago by anonymous

Bumping this. This is a good feature, and there should be a way to work around this issue somehow.

comment:5 Changed 5 years ago by skyper

Well, there are quite some tickets about terracer.

Did not use it lately as it is destroying history and really buggy with associatedStreet relations.

If creating a new relation you need to first check for an already existing relation and then create a proper relation

  • how to handle incomplete data (e.g. only part of one street downloaded)
  • always add at least one member with role street
    • better use minimum all connected ways with the same name (max all ways with same name in this area)

Core already includes some methods for associatedStreet relations (auto role, validation) and the plugin should use the classes if possible.

comment:6 Changed 5 years ago by Don-vip

Resolution: fixed
Status: newclosed

Fixed in [o30611:30615].

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.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.