Opened 18 months ago

Last modified 18 months ago

#22343 new enhancement

Warn about junction=roundabout with oneway=no

Reported by: Famlam Owned by: team
Priority: normal Milestone:
Component: Core validator Version:
Keywords: template_report circular roundabout oneway Cc:

Description (last modified by Famlam)

What steps will reproduce the problem?

Validate attached example (next comment). The northern part of the cycleway is bi-directional (oneway=no). Such a case cannot fulfill the requirements of junction=roundabout (With bi-directional traffic, there must be a "yield" for one of the traffic flows or there will be a conflict between incoming/outgoing traffic and traffic remaining on the roundabout, so the roundabout would never have the complete right of way as required for a junction=roundabout). In this example, the Dutch traffic rules require that counter-clockwise traffic must yield for the traffic coming from the sides).
In addition, it conflicts with the implied oneway=yes.

What is the expected result?

Warn about junction=roundabout being used with oneway=no. Propose to use junction=circular instead

What happens instead?

No warning

Please provide any additional information below. Attach a screenshot if possible.

Proposed rule:

way[junction=roundabout][oneway?!] {
  throwWarning: tr("{0} together with {1}", "{0.tag}", "{1.tag}");
  group: tr("suspicious tag combination");
  suggestAlternative: "junction=circular";

Worldwide 815 cases at the moment.

Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2022-08-29 16:59:00 +0200 (Mon, 29 Aug 2022)
Build-Date:2022-08-30 01:30:57
Relative:URL: ^/trunk

Identification: JOSM/1.5 (18543 nl) Windows 10 64-Bit
OS Build number: Windows 10 Home 2009 (19044)
Memory Usage: 868 MB / 1820 MB (633 MB allocated, but free)
Java version: 1.8.0_341-b10, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM
Look and Feel:
Screen: \Display0 1920×1080 (scaling 1.00×1.00)
Maximum Screen Size: 1920×1080
Best cursor sizes: 16×16→32×32, 32×32→32×32
System property file.encoding: Cp1252
System property sun.jnu.encoding: Cp1252
Locale info: nl_NL
Numbers with default locale: 1234567890 -> 1234567890
Dataset consistency test: No problems found

+ OpeningHoursEditor (35924)
+ SimplifyArea (35978)
+ imagery_offset_db (35978)
+ pbf (35978)
+ pt_assistant (1ff2e15)
+ reverter (36011)
+ tageditor (36011)
+ turnlanes-tagging (v0.0.5)
+ undelete (36011)
+ utilsplugin2 (36011)

Tagging presets:

Map paint styles:
+ %UserProfile%\Documents\tijdelijke bestanden\josm-eigen.mappaint.mapcss

Validator rules:
+ %UserProfile%\Documents\tijdelijke bestanden\josm-eigen.validator.mapcss

Last errors/warnings:
- 00008.803 E: Lokaliseren van afbeelding 'bus.png' mislukt

Attachments (1)

circular_as_roundabout.osm (80.4 KB ) - added by Famlam 18 months ago.
Example containing oneway=no on a junction=roundabout way

Download all attachments as: .zip

Change History (2)

by Famlam, 18 months ago

Attachment: circular_as_roundabout.osm added

Example containing oneway=no on a junction=roundabout way

comment:1 by Famlam, 18 months ago

Description: modified (diff)

Modify Ticket

Change Properties
Set your email in Preferences
as new The owner will remain team.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from team to the specified user.
Next status will be 'needinfo'. The owner will be changed from team to Famlam.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.
The owner will be changed from team to anonymous. Next status will be 'assigned'.

Add Comment

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