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

Add area rendering for amenity=doctors #2519

Closed
wants to merge 2 commits into from

Conversation

kocio-pl
Copy link
Collaborator

@kocio-pl kocio-pl commented Dec 20, 2016

[UPDATE: now only for amenity=doctors]

Both amenity=clinic and amenity=doctors can be tagged as an area and it would be good to show it the same as with hospitals:

1pnjiy6r

amenity=clinic area rendering is half baked, because I had not enough experience when trying to add the icon.

@polarbearing
Copy link
Contributor

This provides some progress with a single pending area rendering requests.

However we are not closer to a decision how to structure the bulk of missing areas systematically in general, #1624.

As it just copies the hospital style, which is thematically close enough, so it does not hinder upcoming general decisions.

@imagico
Copy link
Collaborator

imagico commented Dec 31, 2016

I had a look at this and tag use at the moment indicates there are about 1400 area features tagged amenity=doctors and 2300 amenity=clinic without a building tag (which would be the only ones that would be rendered with the style suggested here).

I would be in favor of this change for amenity=clinic in analogy with hospitals but it does not make a lot of sense for amenity=doctors IMO. Of the 1400 features quite a lot seem to actually represent buildings but without the building tag or ways congruent with mapped buildings. The wiki indicates the tag should be applied to the doctor's office which is always a building or part of it and there is no documentation how to verifiably map an outdoor part of amenity=doctors While it is conceivable that a clinic or hospital has outdoor components (like for patient recreation and rehabilitation) that would make sense to be visibly indicated to belong to it the same does not really make a lot of sense for a doctor's office.

@dieterdreist
Copy link

dieterdreist commented Dec 31, 2016 via email

@dieterdreist
Copy link

dieterdreist commented Dec 31, 2016 via email

@kocio-pl
Copy link
Collaborator Author

kocio-pl commented Dec 31, 2016

A doctor's office is intended as the place where a doctor operates, which will most often be a building or part of it, yes, but it doesn't necessarily require a building.

Not necessarily, of course. If they are in the building, they are already rendered properly and this PR doesn't change this case.

However in Poland there are a lot of local "health centres" with pretty basic set of services - which is in fact just a lot of doctors offices gathered in one building with clear area around:

http://www.openstreetmap.org/way/327763405

I was also thinking about rural places (in Africa for example), where the area of the doctor's office may be known, but the buildings may not be there at all (just tents).

@HolgerJeromin
Copy link
Contributor

If someone has forgotten a building tag this PR would show this missing tag better than the current rendering. Right now no one would think that the poi is an polygon.
So +1 for the PR

@kocio-pl
Copy link
Collaborator Author

kocio-pl commented Jan 2, 2017

I can split this PR, so we could merge the part that raises no protests and discuss the second part further, but of course it'd be easier if the split is just not needed.

@imagico
Copy link
Collaborator

imagico commented Jan 2, 2017

Yes, rendering of amenity=clinic areas seems to be without objections so i would merge it. amenity=doctors in my eyes currently lacks a convincing concept of verifiably mapping a doctor's office as an area independent of the physical rooms it is located in.

@dieterdreist
Copy link

dieterdreist commented Jan 2, 2017 via email

@imagico
Copy link
Collaborator

imagico commented Jan 2, 2017

You are not going to convince me here by pointing to other tagging inconsistencies. Neither is the argument going to fly that a doctor's office can theoretically exist without a building if you cannot point to real life occurrences of this in significant numbers. Note i do not make such assessments as given here as a spontaneous whim - i looked at the tag definition and the mapping practice and formed an opinion based on that. If you want to change my opinion you need to address the points i mentioned. I also wonder if any of you has actually looked at the data on a global scale, i.e. the 1400 features i mentioned and not just a few select examples?

@kocio-pl
Copy link
Collaborator Author

kocio-pl commented Jan 2, 2017

Just for the record:

  1. For me they are not inconsistencies - I just have different interpretation (exactly as @dieterdreist wrote it), which is as good as your interpretation, because there's no hard definition. Maybe landuse=* would be better, but that means rendering dept. trying to redefine tagging practice, which is IMO absolutely not how it should be done.

  2. These examples also show that there are cases where it's valid and my take is it's up to the mappers to use it wisely - rules are the same for every such amenity and I see no reason we should treat this one different. Again - if you want to change them, Tagging list is probably where you should start.

But that means the split is needed anyway and I'll do it shortly.

@imagico
Copy link
Collaborator

imagico commented Jan 2, 2017

To be clear - i respect the different opinions stated here and am not convinced my view is necessarily any better than others.

@kocio-pl kocio-pl changed the title Add area rendering for amenity=clinic and amenity=doctors Add area rendering for amenity=doctors Jan 2, 2017
@kocio-pl
Copy link
Collaborator Author

kocio-pl commented Jan 2, 2017

I leave this PR for amenity=doctors because the discussion is here already.

@pnorman
Copy link
Collaborator

pnorman commented Jan 3, 2017

I can't think of anywhere locally where what's proposed makes sense, for the reasons @imagico mentions above.

@matthijsmelissen
Copy link
Collaborator

There seems to be no support for this proposal so I'm going to close it.

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

Successfully merging this pull request may close these issues.

7 participants