-
Notifications
You must be signed in to change notification settings - Fork 279
Optional tms metadata #1160
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
Optional tms metadata #1160
Conversation
|
This version expects simple space separated keyword_list param in tms md. {{if service.keyword_list and len(service.keyword_list) > 0}} |
|
Thanks for this work! Can you please provide an example configuration with all provided keywords and add it to the documentation? It should work as similar as possible to the wms metadata described here: https://mapproxy.github.io/mapproxy/latest/services.html#full-example Could you also provide a very simple test here: mapproxy/mapproxy/test/system/test_tms.py Line 36 in 5699e22
You only need to provide one metadata key and check for it's appearance in the result, I think that's enough to ensure the basic functionality of this feature. |
|
Added simple test and full example in doc for second approach with standard keyword_list (we would prefer). Should we notice in doc, that if no special service tms md block is provided the wms md entries are used by default? |
|
Yes that would be useful. It should go into a |
Co-authored-by: Simon Seyock <8100558+simonseyock@users.noreply.github.com>
simonseyock
left a comment
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.
Thanks for your contribution!
We would provide optional tms metadata KeywordList and ContactInformation for tms capabilities.
(https://wiki.osgeo.org/wiki/Tile_Map_Service_Specification)
We are doing it with code below since a few years for our customer.
i.e. https://via.bund.de/wsv/bwastr/tms/1.0.0