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

Dangerous condition due mtb:scale=6 rendered like mtb:scale=3 #583

Closed
amadvance opened this issue Jun 23, 2021 · 4 comments
Closed

Dangerous condition due mtb:scale=6 rendered like mtb:scale=3 #583

amadvance opened this issue Jun 23, 2021 · 4 comments

Comments

@amadvance
Copy link

Hi,

At present the mtb:scale=6 attribute is rendered exactly like the mtb:scale=3. This is a potential dangerous condition as people may enter trails impossible to ride. (The 6 exactly means not rideable).

Note that the mtb:scale=6 is an officially approved difficulty level. See: https://wiki.openstreetmap.org/wiki/Key:mtb:scale

The following screenshot show two nearby trails, one mtb:scale=3 and the other mtb:scale=6. You can easily see that they have identical aspect.

mtb_scale_3_vs_6

This is the location of the screenshot: https://www.cyclosm.org/#map=20/45.77212/9.52970/cyclosm

Considering that people may get in serious trouble, whatever solution that remove this hazard is welcome. In case you can just show them as mtb:scale=5, or bycicle=no.

Thanks!
Andrea

@quasart
Copy link
Contributor

quasart commented Jun 27, 2021

Indeed, no style is defined there for scale 6 :

https://github.com/cyclosm/cyclosm-cartocss-style/blob/master/roads.mss#L3606

@Florimondable
Copy link
Member

condition [mtb_scale<6] should be added to the main condition :
#roads_high::mtbscale[mtb_scale>=0][zoom>=15]

[mtb_scale=x] should be also replace by [mtb_scale>=x]

What I don't understand is level S6 si not defined by the STS http://www.singletrail-skala.de/, is it only used by OSM contributors ?

@amadvance
Copy link
Author

The S6 is used to mark segments that you cannot ride, and you have to walk them. It's someway expected that the STS doesn't report it, as they are not rideable.

Phyks added a commit that referenced this issue Jun 30, 2021
@Phyks
Copy link
Member

Phyks commented Jun 30, 2021

Done in a5bb525 with an extra dash for mtb_scale=6.

2021-06-30-234716

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

No branches or pull requests

4 participants