Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Spuriousness "Missing way on level crossing" warnings #1327

Closed
danfos opened this issue Sep 8, 2021 · 5 comments
Closed

Spuriousness "Missing way on level crossing" warnings #1327

danfos opened this issue Sep 8, 2021 · 5 comments

Comments

@danfos
Copy link
Contributor

danfos commented Sep 8, 2021

I see quite some "Missing way on level crossing" warnings but I do not see what is wrong, the node is on a crossing between a way marked as highway=* and railway=*

For examples and further details see:

Or use http://osmose.openstreetmap.fr/en/map/#item=7090&zoom=7&lat=51.113&lon=4.883&level=2&tags=&fixable= to find more examples.

I had a look at https://github.com/osm-fr/osmose-backend/blob/master/analysers/analyser_osmosis_highway_features.py but it is not clear to me where things go wrong.

Does this have anything to do with #1216?

@Bibi56
Copy link

Bibi56 commented Sep 18, 2021

Here I was about to add a ticket for railway=disused but the issue seem to be more generic. crossing are for pedestrians, level_crossing for vehicles.

@marsip
Copy link

marsip commented Sep 20, 2021

Thanks for your answers. I will mark these as false positives in OSMOSE.

@smootheFiets
Copy link

Those false positives crop up in the Netherlands, too. I came across this example:
https://osmose.openstreetmap.fr/en/issue/e7c28be0-063f-c3dc-ae73-ce9fc495f420

I gather from the error message that it finds the corresponding highway=unclasified OK, but it doesn't appear to find the railway https://www.openstreetmap.org/way/396695648 (although the offending node is part of it)

I hope this helps debugging.

frodrigo added a commit to frodrigo/osmose-backend that referenced this issue Dec 27, 2021
jocelynj added a commit that referenced this issue Dec 27, 2021
* frodrigo/master:
  Also keep nature=Sommet for natural=peak #1354
  Remove diff mode in analyser_osmosis_waterway #1348
  Remove diff mode in analyser_osmosis_highway_features #1327
  mapcss: update
  mapcss: more flexible link selector
  Allow '/' in Denmark #1335
@danfos
Copy link
Contributor Author

danfos commented Jan 1, 2022

Thanks for the fix, the spurious warnings are gone and the number of remaining warnings is back on the normal level.

Screenshot_20220101_115705

Checked the remaining warnings in my area and see they are "real" problems

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants