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

Switch on rendering of access tag by default #10983

Open
joostschouppe opened this issue Feb 23, 2021 · 28 comments
Open

Switch on rendering of access tag by default #10983

joostschouppe opened this issue Feb 23, 2021 · 28 comments

Comments

@joostschouppe
Copy link

🚀 feature request

Switch on rendering of access tag by default

The Belgian OpenStreetMap chapter processes more than one complaint per month from people who say a path on their property should not be used by the general public. They generally want us to delete them. But if the path is observable, we explain that it's better to keep it on the map, and (hopefully) correctly map it as private. However, many apps don't show that info clearly on the map. As a community, we're trying to talk to app-makers to take this info in account. Osmand does a great job at showing the info, BUT the feature to show this info is a little bit hidden. The suggestion is to switch this on by default.

So the suggestion is: on new installs, activate the rendering of accesibility by default.

This is of particular interest to hikers and bikers, so perhaps the default could be associated to just those profiles.
However, for cars, the information is becoming more and more relevant too, as more details are getting mapped. The number of driveways worldwide has doubled in the last year or so - and a lot of them are private. Access roads in general can provide useful shortcuts. Users might be annoyed when they are not used by the routing, showing them as private on the map might make it easier to understand routing choices. The same layer also contains toll road info, which is also always nice to know.
For 4WD vehicles the info is also of particular interest.

@vshcherb vshcherb added the Observed Needs more clarification, feedback, or research label Mar 1, 2021
@vshcherb
Copy link
Member

vshcherb commented Mar 1, 2021

I tend to disagree cause it needs to come with explanation. It's 1 story if you enable something and starts to see new dashes / strange lines and completely another if app uses it by default without proper introduction.
If you want to share within specific group it's better to create profile: OpenStreetMap Belgium and share it within group with proper communication.

I would definitely agree that we need to render more icons that could explain but different types of dashes are not very explanatory .

@joostschouppe
Copy link
Author

I understand your point. Though red dots on top of a path are pretty self-explanatory IMHO. To realize blue means destination and purple is with a fee, yeah that does take a little more time.

(note that it is exactly the difficulty of communicating to data users, in this case hikers and bikers, that is the difficult part. Hence why we are talking to different map developers to make it more obvious when a path isn't for general use. Different solutions are possible, like showing them only at high zoom, overlaying access icons, not showing them at all,...)

@vshcherb
Copy link
Member

vshcherb commented Oct 3, 2023

To do:
Icons needs to be clickable (POI)
image

@vshcherb vshcherb added rendering and removed Observed Needs more clarification, feedback, or research labels Oct 3, 2023
@ivanPyrohivskyi ivanPyrohivskyi self-assigned this Oct 5, 2023
@vshcherb vshcherb modified the milestones: 4.7-backend, 4.6-backend Oct 17, 2023
@vshcherb
Copy link
Member

@vshcherb vshcherb added the ! label Oct 24, 2023
@dmpr0
Copy link
Contributor

dmpr0 commented Nov 23, 2023

Review - 2

  • Unable to find No access icons

@dmpr0 dmpr0 reopened this Nov 23, 2023
@johnpercy
Copy link

johnpercy commented Nov 23, 2023

Toll roads seem to be permanently on for me except for low zoom in this commit.
I think UniRS will also need the same treatment as Touring to get access and tolls to work again

@xmd5a2
Copy link
Contributor

xmd5a2 commented Nov 24, 2023

@johnpercy I've checked UniRS and it already works as expected

@xmd5a2
Copy link
Contributor

xmd5a2 commented Nov 24, 2023

@dmpr0 see #10983 (comment)

@ivanPyrohivskyi
Copy link
Contributor

New access icons will be available in the new December 2023 maps.
For test need regenerate map.

@vshcherb vshcherb closed this as completed Dec 1, 2023
@dmpr0 dmpr0 reopened this Dec 4, 2023
@dmpr0
Copy link
Contributor

dmpr0 commented Dec 4, 2023

Review

  • Disable red circles by default, only "Stop" icons should be visible - red circles are fine with more details option on

Screenshot
  • Context menu doesn't provide any relevant information - is it possible to fix ?
  • Next task: support conditional tags?

@vshcherb
Copy link
Member

vshcherb commented Dec 7, 2023

Detection border points to place stop icons:

  1. Most complex and correct part that stop icons shouldn't be placed on the OSM node but not in the middle between OSM Way segments. For example "T"-intersection and "I" is a private road, we shouldn't place stop icon on intersection as it confusing.
  2. Correct detection border point (as OSM node): it's a point that has intersection of "private" and "non-private roads". To get it working correctly all roads need to be analyzed which is very significant performance implication.

xmd5a2 added a commit to osmandapp/OsmAnd-resources that referenced this issue Dec 8, 2023
@vshcherb
Copy link
Member

vshcherb commented Dec 10, 2023

This looks definitely incorrect, so it's a question whether we are ready to release this feature.

@ivanPyrohivskyi ivanPyrohivskyi self-assigned this Dec 13, 2023
xmd5a2 added a commit to osmandapp/OsmAnd-resources that referenced this issue Dec 14, 2023
xmd5a2 added a commit to osmandapp/OsmAnd-resources that referenced this issue Dec 14, 2023
xmd5a2 added a commit to osmandapp/OsmAnd-resources that referenced this issue Dec 14, 2023
xmd5a2 added a commit to osmandapp/OsmAnd-resources that referenced this issue Dec 14, 2023
xmd5a2 added a commit to osmandapp/OsmAnd-resources that referenced this issue Dec 14, 2023
xmd5a2 added a commit to osmandapp/OsmAnd-resources that referenced this issue Dec 14, 2023
xmd5a2 added a commit to osmandapp/OsmAnd-resources that referenced this issue Dec 14, 2023
@xmd5a2
Copy link
Contributor

xmd5a2 commented Dec 14, 2023

All commits were reverted

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

9 participants