Modify

Opened 2 years ago

Closed 22 months ago

#22831 closed enhancement (fixed)

[PATCH] Expand historic=memorial preset subkey memorial=*

Reported by: JeroenHoek Owned by: team
Priority: normal Milestone: 23.05
Component: Internal preset Version:
Keywords: Cc:

Description

Adds stolperstein, ghost_bike, blue_plaque, cross, sculpture, and bench.
Give all of them descriptions (useful for words like 'stele').

All of these have over 1,000 uses, with the exception of bench and ghost_bike which have several hundred, but are well documented on the wiki.

(I noticed sculpture was missing, and updated the list judging from current use.)

Attachments (1)

0001-Memorial-preset-descriptions-and-extra-values.patch (2.9 KB ) - added by JeroenHoek 2 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 by taylor.smock, 23 months ago

Milestone: 23.05

I kind of wish we had a regions tag for presets (see vespucci). I don't like the (United Kingdom)/(Europe, Russia) prefixes in the tagging. We do have (<COUNTRY CODE>) elsewhere, but it is for crossing_ref, payment:*, and ref:*. Those locations use ISO 3166 country codes, but I would prefer to avoid adding more.

Other than that, it looks good.

@JeroenHoek: How would you feel if I dropped the blue_plaque and stolperstein for now? I would expect both of those to be regionally limited and not widely used.

in reply to:  1 ; comment:2 by JeroenHoek, 23 months ago

Replying to taylor.smock:

I kind of wish we had a regions tag for presets (see vespucci). I don't like the (United Kingdom)/(Europe, Russia) prefixes in the tagging. We do have (<COUNTRY CODE>) elsewhere, but it is for crossing_ref, payment:*, and ref:*. Those locations use ISO 3166 country codes, but I would prefer to avoid adding more.

Other than that, it looks good.

@JeroenHoek: How would you feel if I dropped the blue_plaque and stolperstein for now? I would expect both of those to be regionally limited and not widely used.

Without those two this patch should already improve JOSM, so that's fine if you dislike those. It would be nice to have a way to include them sensibly though; Stolpersteine are quite common in Europe.

For reference: stolperstein has 9200 uses, blue_plaque 1100.

in reply to:  2 comment:3 by taylor.smock, 23 months ago

Replying to JeroenHoek:

For reference: stolperstein has 9200 uses, blue_plaque 1100.

I hadn't checked either in taginfo. stolperstein is pretty close to the JOSM thresholds (10k), so I'll go ahead and leave both in. I'll just modify the country references ((GB) and (EU,RU) I think).

comment:4 by JeroenHoek, 23 months ago

I'll just modify the country references ((GB) and (EU,RU) I think).

Note that blue plaques are used in all of the United Kingdom, not just Great Britain (GB). They are used in Northern Ireland as well.

in reply to:  4 comment:5 by taylor.smock, 23 months ago

Replying to JeroenHoek:

Note that blue plaques are used in all of the United Kingdom, not just Great Britain (GB). They are used in Northern Ireland as well.

You might want to checkout wikipedia:ISO_3166-2:GB. :)

comment:6 by taylor.smock, 22 months ago

Resolution: fixed
Status: newclosed

In 18722/josm:

Fix #22831: Add additional memorial values for the historic=memorial preset (patch by JeroenHoek, modified)

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.