Modify

Opened 7 weeks ago

Closed 7 hours ago

Last modified 7 hours ago

#24070 closed defect (fixed)

Allow tracktype=grade2 together with surface=ground

Reported by: Nekzuris Owned by: team
Priority: normal Milestone: 25.02
Component: Core validator Version:
Keywords: tracktype grade2 ground Cc: mkoniecz, skyper

Description

I don't think surface=ground is incompatible with tracktype=grade2.

According to the wiki, grade2 is mostly solid and without grass in the middle.
The appropriate surface is often compacted but that is always a bit subjective, I use it when material has been added and worked on by machine, but sometimes the road is simply the natural ground that has been naturally compacted over time with regular use.
For that I use surface=ground because it's not dirt and I think it's more precise than unpaved, it's also not grade3 because there is no grass in the middle.

https://github.com/osm-fr/osmose-backend/issues/2427

This warning comes from here:
https://josm.openstreetmap.de/browser/josm/trunk/resources/data/validator/combinations.mapcss#L348
And I would also remove ice and salt.

Attachments (0)

Change History (12)

comment:1 by stoecker, 7 weeks ago

Comes from #22102.

comment:2 by stoecker, 7 weeks ago

Reviewing the tags I agree that ground, ice and salt may be supported for type 2 even if uncommon.

So I don't think we should warn in these cases.

Last edited 7 weeks ago by stoecker (previous) (diff)

comment:3 by stoecker, 7 weeks ago

Cc: mkoniecz skyper added
Milestone: 25.01

Other Comments?

comment:4 by GerdP, 7 weeks ago

My understanding is that type 2 means something like "not paved" and "no soft material". surface=ground can be anything from very soft to hard like stone.

comment:5 by stoecker, 7 weeks ago

So surface=ground + tracktype=grade2 would specify ground, but not soft. And thus it is a valid combination I think, or?

comment:6 by GerdP, 7 weeks ago

Yes, I agree. Not so sure about ice and salt. My understanding is that both can be soft in rainy conditions.

comment:7 by stoecker, 7 weeks ago

I assume they are valid in regions like poles (ice) and deserts (salt).

comment:8 by Nekzuris, 7 weeks ago

Good that we agree on ground + grade2.
I'm not sure about salt, but I don't know how you can have soft ice. But most ice roads are seasonal and tagged based on the material underneath, often gravel or compacted.

comment:9 by stoecker, 3 weeks ago

Milestone: 25.0125.02

Ticket retargeted after milestone closed

comment:10 by anonymous, 3 weeks ago

From the basics of the proposal for track_type to its current use, everything should actually speak against this combination. In the proposal, it was stated for Grade2 in quote “A track that is heavily used, and has stones/gravel”. It was also made clear in the discussion at the time that a Grade2 Way “is covered by stone to ease its use by cars”. The German Wiki uses the term “water-bound surface with a load-bearing substructure (compacted gravel or other materials)" / Original "Wassergebundene Decke mit tragfähigem Unterbau (Verdichteter Schotter oder andere Materialien)".
When we speak of surface=ground, we generally mean a natural ground that has not been altered by human hands. This should be at least a grade 4, or even better a grade 5, but never a grade 2. If the road is so good that it qualifies for a grade 2, then surface=ground is simply wrong.

comment:11 by stoecker, 7 hours ago

Resolution: fixed
Status: newclosed

In 19337/josm:

fix #24070 - no longer warn for surface=ground + tracktype=grade2

comment:12 by stoecker, 7 hours ago

@anonymous

I believe your assumptions are only true for e.g. European conditions. But we should not warn for anything which can be used somewhere else. So if there is reasonable doubt, then the warning should not be issued.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.