-
Notifications
You must be signed in to change notification settings - Fork 164
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
marina preset: do not suggest to add unnecessary duplicative seamark tags #899
Conversation
seamark tagging scheme in general creates unneeded parallel tagging schema, there is no need to support it an bother user about it
🍱 Preview the tagging presets of this pull request here: https://pr-899--ideditor-presets-preview.netlify.app/id/dist/#locale=en. |
For reference: these tags were added in openstreetmap/iD#7622. See also #683 My first impression is that while potentially duplicating some information, the However, from what I can see, Footnotes
|
It would be a real shame to stop supporting With this argument, should the warnings to add There's a lot of things i don't like about seamark tags, but it's become well established and this is one of the very few cases where there is an equally popular tag in the normal tag system... There are very few marinas in the database, even in big coastal cities, so has anyone actually found this specific warning annoying? |
To make it clear: this was not meant to stop supporting the seamark tags altogether. Only for specific cases (like for marinas), where their value could be implied by another tag might be "considered redundant". But I think you have a very good point: Like in the healthcare case, we should consider the use-case specific tags (here: You convinced me: I think, in general, keeping this as a proper tag upgrade does actually make sense to keep. Any feedback on the |
So, to be clear, leisure=marina doesn't have a 1:1 relationship with a seamark tag. seamark:harbour:category=marina means that the facility has services that a captain might be looking for. seamark:harbour:category=marina_no_facilities means that the marina basically has a place to tie up. So, it is proper to have leisure=marina and the seamark at the same time since the seamark is more descriptive which is important to a maritime user. I'm not sure how to automate such a seamark tag. |
In cases where it is not "always implies seamark:XYZ" then such "tag upgrade" (which BTW should be maybe renamed to make clear that it is automated and not very smart replacement) is damaging data.
Well, in cases where there specific tag always indicates some seamark tags then in such case they are pointless duplicates and pure waste of time to add them. seamark: especially are irritating as they appear to basically have parallel tagging schema, with
not really, EDIT: And we do not have |
Is it still I feel that it can encourage adding fake marinas where there is anchorage/berthing only (though maybe it is still a marina?) |
me, that is why I created this PR |
Yes, it's still a marina. A marina implies a place to dock that isn't an anchorage that is open for non-owners to use. If there are facilities (and "facilities" has a wide meaning), then it is still a marina, it just has added amenities. |
You have to take this in context. seamark:small_craft_facility:category=toilets specifically calls out a toilet that is built/maintained for small craft users. It doesn't stop people from using other toilets or for non-boaters from using the toilet, but when you are looking at a sea map, you can see the facilities that are specifically for you (as a boater) or are amenities that are boater friendly. No filtering involved, the map is very simple and easy to look at. |
Yes! I agree that the addition of "no_facilities" is meaningful - especially for sailors planning a route. Certainly in the UK, it is common place for toilet facilities not be for exclusive use of the users of the marinas not the general public so appropriate that they only appear as such in OpenSeaMap. What I do this is a bit odd is the use of the term 'seamark' (presumably as opposed to landmark) for anything that needs to be included in OpenSeaMap that is is not in OpenStreetMap. |
Hm… The not a 1:1 match argument by @matkoniecz is actually a valid critique here. 🤔 On the other hand I would also like to support openseamap tagging in iD wherever reasonably possible, which is made tricky because there does not seem to be a good way to bring the openseamap tags perfectly in line with the "standard" osm tags. It is also a bit unfortunate is that the What about the following solution: We keep the current tag upgrade of the Footnotes
|
tyrasd commented that a better description of which "extra facilities" are expected would be quite useful. I interpret this as a request to clarify what facilities should be mapped in OpenSeaMap using seamarks. In thinking about this I do think that mappers should realise that sailors and other seafarers typically have cruising guides, pilot books, almanacs and other documents that they can refer to when planning passages. These documents already provide details of the facilities that are available - albeit in different ways! One popular publication is Reeds Nautical Almanac.
For each harbour or marina a paragraph entitled Facilities: The symbols used for every harbour and marina covered within the 1000+ pages: Other publications provide similar info. Often in much less detail than Reeds. It seems to me that in relation to facilities, seafarers need to know:
I hope the above is of some help in this discussion NB. I will add some images later to illustrate |
I think this would be a good solution. |
seamark tagging scheme in general creates unneeded parallel tagging schema, there is no need to support it an bother user about it