Modify

Opened 2 months ago

Closed 2 months ago

#20089 closed defect (fixed)

Don't flag railway=rail crossing railway=yard

Reported by: GerdP Owned by: GerdP
Priority: normal Milestone: 20.12
Component: Core validator Version:
Keywords: template_report Cc:

Description

What steps will reproduce the problem?

  1. Download area around https://www.openstreetmap.org/way/266945129
  2. run validator

What is the expected result?

no warnings about Crossing railways

What happens instead?

Crossing railways (8)

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

Build-Date:2020-11-15 10:37:11
Revision:17310
Is-Local-Build:true

Identification: JOSM/1.5 (17310 SVN en) Windows 10 64-Bit
OS Build number: Windows 10 Home 2004 (19041)
Memory Usage: 758 MB / 1753 MB (389 MB allocated, but free)
Java version: 1.8.0_272-b10, AdoptOpenJDK, OpenJDK 64-Bit Server VM
Look and Feel: com.sun.java.swing.plaf.windows.WindowsLookAndFeel
Screen: \Display0 1920x1080 (scaling 1.0x1.0)
Maximum Screen Size: 1920x1080
Best cursor sizes: 16x16 -> 32x32, 32x32 -> 32x32
VM arguments: [-agentlib:jdwp=transport=dt_socket,suspend=y,address=localhost:52777, -ea, -javaagent:D:\eclipse-java-2020-09\eclipse\configuration\org.eclipse.osgi\215\0\.cp\lib\javaagent-shaded.jar, -Dfile.encoding=UTF-8]
Program arguments: [--debug]
Dataset consistency test: No problems found

Plugins:
+ buildings_tools (35640)
+ o5m (35640)
+ pbf (35640)
+ poly (35640)
+ reverter (35640)
+ undelete (35640)
+ utilsplugin2 (35640)

Validator rules:
+ c:\josm\core\resources\data\validator\geometry.mapcss

Attachments (2)

20089.patch (651 bytes) - added by GerdP 2 months ago.
20089.2.patch (712 bytes) - added by GerdP 2 months ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 2 months ago by GerdP

Owner: changed from team to GerdP
Status: assignednew

Changed 2 months ago by GerdP

Attachment: 20089.patch added

comment:2 Changed 2 months ago by GerdP

with patch the test ignores crossings of ways if one is railway=yard and the other is not

Changed 2 months ago by GerdP

Attachment: 20089.2.patch added

comment:3 Changed 2 months ago by GerdP

Make sure both ways have railway=* tag. First patch suppressed also warnings about highway=* crossing railway=yard,

comment:4 Changed 2 months ago by Don-vip

Milestone: 20.1120.12

Milestone renamed

comment:5 Changed 2 months ago by GerdP

Resolution: fixed
Status: newclosed

In 17348/josm:

fix #20089: Don't flag railway=rail crossing railway=yard

  • ignore crossings of ways if one is railway=yard and the other is not and both have the railway key

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain GerdP.
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.