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

adding medical specialities #408

Closed
matkoniecz opened this issue Feb 6, 2023 · 12 comments
Closed

adding medical specialities #408

matkoniecz opened this issue Feb 6, 2023 · 12 comments

Comments

@matkoniecz
Copy link

Not exactly sure what kind of changes you want but picking up matkoniecz@ca3ecde (adding medical specialities) may be useful if you or other users would report missing specialities in iD presets and/or contribute translations to iD presets in their languages, making this quest more ready for wider release and testing is it even a really good idea.

Note that you can add only one at time, so multiple ones would require making a note.

So far as result of testing this code I opened openstreetmap/id-tagging-schema#778 openstreetmap/id-tagging-schema#779 https://www.openstreetmap.org/changeset/132128903 https://www.openstreetmap.org/note/3546322 (pain management - it seems we have no tag for that) when I tried to use it

(StreetComplete issue is at streetcomplete#1020 )

@Helium314
Copy link
Owner

Helium314 commented Feb 6, 2023

This would definitely be useful, though I think I'll adapt the quest to fit into the current healthcare speciality quest (and allow switching to the current UI for entering multiple values).

First thing noticed missing: dermatovenereology

@matkoniecz
Copy link
Author

ooo, you have code for that? Is it problematic in some way or unfinished (if it is working well then feel free to make SC PR!)?

@matkoniecz
Copy link
Author

First thing noticed missing: dermatovenereology

If you are sure that it should be added and it has OSM tag then reporting it at https://github.com/openstreetmap/id-tagging-schema/issues would be the best idea (or opening PR there)

@Helium314
Copy link
Owner

Helium314 commented Feb 6, 2023

Another thing is that it might be good to allow searching in language of the current country, so even if you're somewhere else you'd just need to enter what you see on the sign.
But it looks like the feature dictionary doesn't support that (or I'm doing it wrong).

@matkoniecz
Copy link
Author

matkoniecz commented Feb 6, 2023

But it looks like the feature dictionary doesn't support that (or I'm doing it wrong).

Note opening hours days of week display where SC got such feature (after field testing in Italy during STOTM)

even if you're somewhere else you'd just need to enter what you see on the sign.

That seems quite risky due to some terms being extremely close in writing and having drastically different meanings, to say nothing about other mistakes. In such case I would advise to skip such quests.

@westnordost
Copy link
Collaborator

westnordost commented Feb 6, 2023

First thing noticed missing: dermatovenereology

What's that, a doctor that specializes in .... German: "Sexuell über die Haut übertragbare Krankheiten"?
Maybe this field exists, but so do 1000s more. I recently added presets for common physician specializations in openstreetmap/id-tagging-schema#718 but dermatovenereology doesn't sound common.

(I added those as presets, not as translatable fields)

@Helium314
Copy link
Owner

Maybe this field exists, but so do 1000s more

At least in Austria, "Facharzt für Haut- und Geschlechtskrankheiten" is rather common. Not sure, but maybe it's even some kind of default for dermatologist training.

@westnordost
Copy link
Collaborator

westnordost commented Feb 6, 2023 via email

@Helium314
Copy link
Owner

Apparently this is connected also in Gemany: https://flexikon.doccheck.com/de/Dermatologie (Facharztweiterbildung), and wikipedia claims that "In many areas of the world, the specialty is usually combined with dematology".

So I'm not sure now whether the distinction between dermatology and dermatovenereology really matters. Probably only in the some areas of the world.

@westnordost
Copy link
Collaborator

westnordost commented Feb 8, 2023

Also, I wonder if then it shouldn't be two specialities? dermatology; venereology ... anyway, I haven't had a look at your implementation, but that it is valid to have several specialities is something to consider.

Question: For your implementation, do you use the translatable field healthcare:speciality or the preset name? Or both? I think there are not aliases/terms for field values, which means that if you search for "Haut" in German you won't find dermatology. On the other hand, the list of translatable fields for healthcare:speciality in the iD presets is larger, because in my recent PR I only added the most common specialities as own presets.
In any case, if you use the translatable fields, how do you do it? IIRC https://github.com/westnordost/osmfeatures/ does not support (translatable) fields yet (PR welcome, hehe).

@Helium314
Copy link
Owner

Helium314 commented Feb 8, 2023

Also, I wonder if then it shouldn't be two specialities?

Maybe it should actually be both, yes. I wouldn't consider this as a single one, if it didn't already exist in the wiki.

Until last version, the quest asked for plain OSM tags in an EditText, allowing only to enter tags from a pre-defined list (with suggestions), and multiple values.
With 3204e88 it uses the UI as it is currently planned for streetcomplete#1020 (plus optional fallback to old UI for entering multiple values).
Basically it's the default SearchFeaturesDialog, with filtering for doctors. Here you can enter skin and it will find Dermatologist. Works similar in German.

@matkoniecz
Copy link
Author

Though my interface works only when things are defined as features (which triggered my iD issue - it has bad effects for iD mappers anyway so can be fixed also there) and still does not support multiple values.

I wanted to check is there any unexpected blocker before starting fight with RecyclerView.

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

No branches or pull requests

3 participants