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

add validation for connecting highway with extension #5646

Merged
merged 1 commit into from
Jan 9, 2019
Merged

add validation for connecting highway with extension #5646

merged 1 commit into from
Jan 9, 2019

Conversation

gaoxm
Copy link
Collaborator

@gaoxm gaoxm commented Dec 21, 2018

but avoid connect when the edge goes in parallel to the other road.
also added tests for the validation.

@kymckay
Copy link
Collaborator

kymckay commented Dec 22, 2018

I believe it should be re-worded to something along the lines of "Almost Junction" as commonly found in QA tools. Reason being that Potential connection with extension doesn't sound like an error, but rather a suggestion.

Same for the tooltip, it seems to be suggesting a connection should be made when really the user should investigate more closely and either mark with noexit (if there truly is no connection) or map whatever missing connections there are.

@bhousel bhousel requested a review from quincylvania December 22, 2018 18:15
@gaoxm
Copy link
Collaborator Author

gaoxm commented Dec 23, 2018

I believe it should be re-worded to something along the lines of "Almost Junction" as commonly found in QA tools. Reason being that Potential connection with extension doesn't sound like an error, but rather a suggestion.

Same for the tooltip, it seems to be suggesting a connection should be made when really the user should investigate more closely and either mark with noexit (if there truly is no connection) or map whatever missing connections there are.

Makes sense. @bhousel @quincylvania would you guys also suggest a better name/message for this? JOSM calls this "way ends near another highway": https://wiki.openstreetmap.org/wiki/JOSM/Validator

@gaoxm
Copy link
Collaborator Author

gaoxm commented Jan 4, 2019

@bhousel @quincylvania ping - do you think I should update the name and message for the validation check before merging this PR?

@bhousel
Copy link
Member

bhousel commented Jan 5, 2019

Hey @gaoxm - Yes the KeepRight version of this check looks like:

          title: 'Almost junction'
          description: 'This node is very close but not connected to way {var1}.'

I agree with @SilentSpike , that i like the name "Almost Junction" a bit better.

but avoid connect when the edge goes in parallel to the other road.
also added tests for the validation.
@gaoxm
Copy link
Collaborator Author

gaoxm commented Jan 8, 2019

@bhousel @quincylvania validation name and messages are all updated.

@gaoxm gaoxm merged commit 1d1b117 into openstreetmap:validation Jan 9, 2019
@gaoxm gaoxm deleted the connectWithExtend branch January 9, 2019 19:34
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.

3 participants