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

Highway Intersection Check #664

Merged
merged 11 commits into from
Mar 29, 2022
Merged

Conversation

brianjor
Copy link
Collaborator

Description:

Picked up where #643 left off.

Changes are:

  • Flagging the highway and the node(s) that intersects with the powerline/waterway instead of the powerline/waterway
  • Flags the way instead of just the edge

Test Results:

ISO Total Flags Sampled Sampling % TP FP False Positive Rate
ARG 10 10 100% 10 0 0%
AUS 9 9 100% 9 0 0%
TUR 18 18 100% 18 0 0%

Copy link
Collaborator

@Bentleysb Bentleysb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd be interested to see some examples of the true positive flags for this check. I am surprised there are Edges with power line or waterway tags. Those features generally become Lines.

@brianjor
Copy link
Collaborator Author

brianjor commented Mar 15, 2022

I'd be interested to see some examples of the true positive flags for this check. I am surprised there are Edges with power line or waterway tags. Those features generally become Lines.

Here is an example of a highway that has the power=line tag on it that also intersects another highway.
image
The selected way has both highway and power and intersects the "Ruta Nacional 156"
The way "Ruta Nacional 156" is the way that gets flagged.
https://www.openstreetmap.org/edit#map=17/-32.22399/-63.17634

In this example a bunch of highways intersect a highway + waterway (109263731)(https://www.openstreetmap.org/edit#map=18/-28.44676/-65.78058

It looks like the main fix to most of the flagged issues is to separate the way into highway and the waterway/power.

@sonarcloud
Copy link

sonarcloud bot commented Mar 15, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

92.4% 92.4% Coverage
0.0% 0.0% Duplication

Copy link
Collaborator

@Bentleysb Bentleysb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Thanks for the example.

@dmesler2 dmesler2 merged commit 1513bbf into osmlab:dev Mar 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants