-
Notifications
You must be signed in to change notification settings - Fork 261
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
Implement JOSM attributes #815
Conversation
See https://josm.openstreetmap.de/ticket/17285#comment:15 `mirror` is still missing.
While I believe having equivalent attributes in ELI and JOSM is a good idea, could we confirm that these are all actually in use in the JOSM side and not things that were introduced for a specific case and then never reused? |
it is the trend to request "confirmation of use" before a reconciliation :( ping @stoecker @Klumbumbus for the ack |
"transparent" and "format" are relatively new and currently no wms_endpoint entries use them. But they are needed for proper support of wms_endpoint. "custom-http-headers" usually will probably be used extremely seldom for "official" services, that's more a feature to get some strange services to work. It is needed from time to time, but at least I'm always happy if it works without. "last-check" is used, but not as active as it should be - I'm not sure, maybe that idea is a failure and obsoleted by the wiki preview and the automatic checks we do nowadays. Everything else is actively used. |
Just to avoid us adding something, potentially adding support for it in code and JOSM saying "this hasn't really worked" ... definitely asking for a quick "OK" is sensible. |
Regarding last-check: https://josm.openstreetmap.de/ticket/18926 |
last-check is gone now. We have better (i.e. working :-) mechanisms now to server the intended purpose to check the validity of the dataset. |
Oh great, now you've broken my PR :-) |
Merging this as it is. I'll admit the limits of my intellect when it comes to having the mirror tag in the json schema, and happily take any PRs. |
As I understand it that is simply an alternative site for the same data, so shoud be modeled as a json object mirror containing some of the fields with different values. What needs to be checked if one of these objects is enough or if we need an array of them. |
Yep. |
Some like https://josm.openstreetmap.de/wiki/Maps/Czech%20Republic#PrahaIPRlow-vegetationorthophoto have two mirrors. |
You forgot |
I did. |
}, | ||
"valid-georeference": { | ||
"description": "Set to `true` if imagery source is properly aligned and does not need imagery offset adjustments. This is used for OSM based sources too.", | ||
"type": "string" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why not boolean?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Uh huh. Why not!
See https://josm.openstreetmap.de/ticket/17285#comment:15
mirror
is still missing.