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 a category attribute like JOSM, thanks @simonpoole #733
Conversation
👍 Is there any guidance on when a imagery becomes historical? Is historical imagery intended for the ELI? It would be nice to have a description of each somewhere in the documentation, if possible. |
https://josm.openstreetmap.de/wiki/Maps so it isn't actually "historic" the way JOSM uses it, just outdated. |
Any opinions on if we should follow JOSM on this specific point or not @grischard & @Marc-marc-marc ? |
We should follow JOSM on this point. Editors might of course want to show a better explanation - "newer layer available" for example. |
I'm all in favour of making the attribute mandatory, and doing a great big retro-tagging of all existing sources. We should also adapt the conversion scripts. |
as said in the issue, |
I went through the Swiss sources and tried to categorize those with category photo that made sense, and it clearly showed the problem with historicphoto: is a 10 year old layer historic if there is no successor or is it considered current? The remaining question is if we should copy the attribute values from JOSM mechanically, or simply wait till people to it of their own accord? |
if it's the lasted, it'sn't historic for josm.
an automatic bidirectional synchronization would obviously be the best, so that the contribution of the 2 projects would benefit everyone. |
See #136 - JOSM recently added 'elevation' as a type. |
I updated the JOSM database with the new category types: 19 qa and 30 elevation |
20 qa now. I added French BANO as qa too. |
Currently the schema doesn't require category or else we won't be able to build till this is added to ally sources.
ccdc875
to
8ab73de
Compare
I've added it too now, even though it seems to be a slightly weird choice. |
Seems appropriate to me. Though I noticed JOSM actually describes what these values are to be used for and I think we need to do that here too eg.
|
That was updated: https://josm.openstreetmap.de/changeset/15685/josm |
Looks ready to merge for me; we can update the various scripts in another step. |
PS: @andrewharvey I would have used "terrain" |
I think either is fine, so long as we include the description within the schema documentation. |
As discussed.
I'll start adding the field to some of the sources in a 2nd commit.