#5983 closed defect (fixed)
do not render incline like oneway
Reported by: | skyper | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | Internal mappaint style | Version: | latest |
Keywords: | oneway incline | Cc: |
Description (last modified by )
It is missleading if incline is rendered the same way oneway is rendered.
What do you render if a street goes downhill as oneway street ?
Attachments (1)
Change History (17)
comment:2 by , 13 years ago
Priority: | major → blocker |
---|
This is a big bug and the methode is broken for incline anyway. Please stop rendering incline !!!!
Have a look at the attached file if you do not belief. The only two oneway ways are on the left side. I had to revert all ways with negative incline-values to show to be rendered.
comment:3 by , 13 years ago
I wounder how blockers are treated. Shouldn't these tickets be fixed before releasing a new tested version ?
follow-up: 5 comment:4 by , 13 years ago
Priority: | blocker → normal |
---|
follow-up: 13 comment:5 by , 13 years ago
Replying to stoecker:
Hey, Dirk.
I will not change priority but I think this is at least a major bug.
- it is not possible to render incline and oneway within one thread.
- The methode for incline is broken anyway.
Please, just stop rendering incline and we can change this ticket into a normal enhancement.
Thanks a lot.
comment:6 by , 13 years ago
This behaviour is there probably for 4 years, so it can't be a major bug. If I remember correctly it even was a request. So here different views exist. Maybe we need to change it, but it is in no way a major issue.
comment:7 by , 13 years ago
Agree with this being a problem. A different arrow style should be used.
comment:8 by , 13 years ago
The way I know it from paper maps, incline is often displayed with a large thin head of an error >
with up to three in a row to show the degree of incline.
But I think that incline is quite difficult because it changes quite often and with detailed data you have to group and calculate some average to render well, otherwise you will end up to have it only rendered in high zoom levels. (Have a look at my example.)
In a matter of facts the grouping will be needed for oneway, too.
comment:9 by , 12 years ago
Description: | modified (diff) |
---|
#7588 is related. The other "direction keys" (oneway? | aerialway=* | waterway=stream | waterway=river | waterway=canal | waterway=drain | waterway=rapids | "piste:type"=downhill | "piste:type"=sled | man_made="piste:halfpipe" | junction=roundabout
) are directly related to the "flow" of/on this object, but incline
is not. Therefore, I also favour to remove incline
from the "direction keys".
For those (few) who are interested in mapping incline
s, a custom style is probably more useful.
Replying to stoecker:
If I remember correctly it even was a request. So here different views exist.
I couldn't find a corresponding ticket on Trac …
follow-up: 11 comment:10 by , 12 years ago
To (probably) resolve this issue: Are there objections to remove incline from the "direction keys"? :-)
follow-up: 12 comment:11 by , 12 years ago
I couldn't find a corresponding ticket on Trac …
You also need to check josm-dev, talk.de lists, the web forum and my personal e-mail folder :-)
To (probably) resolve this issue: Are there objections to remove incline from the "direction keys"? :-)
Not from my side.
comment:12 by , 12 years ago
Replying to stoecker:
I couldn't find a corresponding ticket on Trac …
You also need to check josm-dev, talk.de lists, the web forum and my personal e-mail folder :-)
I won't :-). As written above, in my opinion, incline should not be mixed with the other direction keys.
If someone needs incline to be rendered as oneway, the preference keys tags.direction
, tags.reversed_direction
can be edited. Maybe, a custom style (say, positive incline: red, negative incline: green) is better suited for extensively mapping inclines.
comment:13 by , 12 years ago
Replying to skyper:
Please, just stop rendering incline and we can change this ticket into a normal enhancement.
My opinion did not change over the last year.
comment:16 by , 8 years ago
There are now atleast two style, which render incline:
- Styles/Incline
- Wheelchair Routing Attributes
by the way incline=-1 or (-8) are rendered wrong and direction=up/down is nort rendered at all