-
Notifications
You must be signed in to change notification settings - Fork 819
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 rendering for healthcare tag scheme #3498
Conversation
@jragusa Please add also |
@Tomasz-W fixed |
@jragusa Just one more thing: I'm wondering if Edit: Are all of |
Have you tested it next to a named retail area and next to trunk roads and
motorways? Those retail name labels are fairly red.
…On Wed, Nov 7, 2018 at 11:21 PM Jérémy Ragusa ***@***.***> wrote:
@Tomasz-W <https://github.com/Tomasz-W> yes I included the healthcare
tags following this comment
<#2938 (comment)>
and they are all displayed as a dot.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#3498 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AoxshLx565SusgwSN2FouU2fDAb3fYjlks5usuyBgaJpZM4YQkcZ>
.
|
@jeisenbe do you have an example of location ? |
|
@jragusa I've read some posts again, and there is a little misunderstanding, sorry for it. I mean to render every |
I think that rendering freeform values is not preferable. With shops it may be acceptable as there are some weird rare shop, but in case of healthcare I would expect that all uses are covered by tagging scheme. |
Here is a TagInfo list of |
@Tomasz-W Sorry, it was a misunderstood from my side. So what's the plan now ? Do I restrict following the TagInfo list or open to any value ( |
@jragusa open to any value (like shops and offices are) |
I would restrict list to the documented values. |
This case is not easy to decide. From one side - shops have much longer list of values and they do not have a global definition, while we have Healthcare 2.0. On the other hand having |
I think that it this case better mapper feedback would be to encourage use of documented values or documenting them if something is missing. While in case of shops I see why rendering |
I think your proposition is perfectly valid for now. We can review this decision later if anyone will complain or just extend the rendering when documentation will be extended. Rendering healthcare is a big step ahead in itself, so we may stay on the safe side and see how it works. |
So, I restrict following the TagInfo list for now ? |
There are still |
I made tests for chiropractor and dialysis. It now work fine. Sorry, for this trivial error, sometimes, I have my head in the clouds. |
A lot of work, but I think it was worth all the hassle. Thanks for your dedication and patience! I think we will come back to this namespace sooner or later, I'm not only sure which one is more true in this case. |
Can someone tell me why the icon for the Charité has no name on the map? |
Most probably because of the road labels and road arrows. |
See also #2429. |
I give up. I've already changed a lot but the conflict persists rendering the name of Charité. |
I would be surprised if anything would change without solving problem I mentioned lately. If you could try to solve it, the chances are high in my opinion. Anybody else interested in fixing this? |
Personally I think its miss-tagged. Things there don't seem to fit the Wiki definition of a hospital. For instance the Institute of Chinese Life Sciences, which is north of the icon but still within the hospital bounds. You might try mapping the area as a university. Since that's what it actually is, even though some places in it provide patent services, and then applying the hospital tag to a more specific area like a building or something. Maybe university would render before hospital. So the name would display. That's what I did for the college near where I live, that also provides dental services and things through internships and it seems to work. I don't think your going to get the name of the hospital to display otherwise. considering how many other things are rendering inside the area. I think its stretching the definition of a hospital a lot as it is anyway (I know from research that a lot of the services it provides would probably be under the amenity=clinic tag instead). Some places like haritéCentrum 15 für Neurologie, Neurochirurgie und Psychiatrie might qualify as social facilities for people with mental health problems also. |
Which is tagging for the renderer. Certainly Charité is both a hospital and (part of) a university, but the main purpose for its existence is that it is a hospital (incorporating education). And for the other point: Most hospitals also provides amenity=clinic type services. |
Its not tagging for the renderer if its miss-tagged in the first place and tagging it properly also happens to get the name to display. I wasnt suggesting just picking random tags until one worked. Which would qualify as tagging for renderer. The buildings should be properly tagged either way. Its missleading to just tag the whole area as a hospital and call it a day when it provides a bunch of other services that can also be tagged. Plus, I noticed the whole area is tagged as emergecy=yes and wheel_chair=yes. surely some of the buildings provide emergecy services and some of the compound is wheel chair friendly, but I doubt the whole does or is. Anyway, its not a tagging disscussion. Except in the case where tagging might possibly be causing the problem. So it really doesnt matter outside of that. |
@Adamant36: even if we would tag Charité as university: why should it be rendered differently than a university and be hidden behind "road labels and road arrows" while the university is not. Science before road labels before charity? BTW: I don't think there is a clear sceme for tagging hospitals linked to universities. If I look at the Universtitätsmedizin Mainz here around the corner: to most people it's clearly a hospital first of all. For students of medicine it will be the university. Either way: neither tagging the area as a hospital with university nodes/smaller areas or a university with hospital nodes/smaller areas inside will be correct. It's all mingled, so one has to decide for one (whatever). But this doesn't help solving the question why there is no text lable for Charité. |
What about |
The university is certainly not the reason. The campus area is displayed correctly and does not overlap. Of course, the students go to the hospital. |
@geozeisig, there's a part here where the hospital grounds almost overlaps itself and there's a whole in it with a weird yellow line going south a bit. It might have something to do with that. There was a hospital a while back in the bay area somewhere where the icon displayed a few yards away from the hospital in a residential area and the name didn't rendering. It was because of a similar issue. I was never able to fix it unfortunately, but if there's not a clear center for the icon and name render or anything overlaps itself it tends to cause rendering problems. So you might check that (I would have fixed it myself but I don't want to screw with the area and the website keeps giving me a loading error for some reason). |
Instances of other places where the hospital name doesn't display correctly (not sure why. Could be rendering priority). https://www.openstreetmap.org/#map=18/37.59184/-122.38181 https://www.openstreetmap.org/#map=18/37.67947/-122.47399 (only doesn't renders name at z18. Might be road oneway arrow blocking it) https://www.openstreetmap.org/#map=18/37.37604/-122.06447 (doesn't render name at z17. Not sure why). https://www.openstreetmap.org/#map=17/37.33501/-121.99849 (It blocks itself out in this situation. Probably miss-tagging). https://www.openstreetmap.org/#map=19/37.33342/-121.88524 (another example of where the name is being blocked out by other stuff due to rendering priority. In this instance a post office, but zoom out). I guess road labels take rendering priority over POI's in most cases? |
Fixes #2938
Changes proposed in this pull request:
health-color
to#CC0000
#BF0000
amenity/healthcare=dentist
to z18 and display a dot at z17amenity/healthcare=clinic
to z17amenity=pharmacy
if tagged in combination withhealthcare=*
Test rendering with links to the example places:
https://www.openstreetmap.org/way/337002580 (new hospital icon and colour)
Before
After (fix correct picture)
https://www.openstreetmap.org/#map=17/46.19317/6.23683 (zoom level and dots)
z17
Before
After
z18
Before
After