forked from mapproxy/mapproxy
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Concurrent instances for mapnik rebased 2023 #3
Open
ArneBab
wants to merge
103
commits into
master
Choose a base branch
from
concurrent-instances-for-mapnik-rebased-2023
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add support for time ranges and miscelaneous Fix default behavior in WMS demo Fix error en exception handling go back to last version in master for forward_req_params Using FileCache instead of DummyCache for dimensions Move dimension handling to the FileCache level to be reused across cache structures Fixing wmts test, disable storage when it is true adding dimensions to wms service Fixing TypeError: is_cached() got an unexpected keyword argument 'dimensions' delete prints Adding ISO 8601 Time intervals Adding ISO8601 Error
…g previous isoformat
Addressed outstanding code review comments
I really bumped my head hard on this issue mapproxy#490 which requires to use the older layer syntax. Further the deprecated example uses source: (singular) for sources key which further makes even trying to use the deprecated syntax by following the example given hard. This PR tries to make things a little clearer.
One more pluralisation fix
Allow for datetime to be a single value
Extend tests and address pr comments
Avoid merge conflicts
Dimensions fixes
…roxy#572) A side effect of PR mapproxy#572 was the following exception when rendering /demo ``` File "/home/even/spatialys/cnes/2022/mapproxy/mapproxy/util/ext/tempita/__init__.py", line 288, in _eval value = eval(code, self.default_namespace, ns) File "<string>", line 1, in <module> File "/home/even/spatialys/cnes/2022/mapproxy/mapproxy/service/demo.py", line 161, in layer_srs sorted_uncached_srs = sorted(uncached_srs, key=lambda srs: get_epsg_num(srs)) TypeError: '<' not supported between instances of 'NoneType' and 'int' at line 59 column 39 in file /home/even/spatialys/cnes/2022/mapproxy/mapproxy/service/templates/demo/demo.html ``` This PR corrects this and adds a minimal test/system/test_demo.py
…le to deal with HEAD requests)
…urces and services
Minor typo updates
Remove 'six' from dependencies
doc: minor typo updates to configuration.rst
request legend in json format
…-found Fix find library not found for ubuntu 22.04
updating because of troubles when doing apt install with non reachable mirrors
This removes python 3.6 from the test matrix as its not supplied by the latest ubuntu image (22.04)
Reverting ubuntu upgrade of action runners
This sidesteps missing theadsafety in mapnik, because the threadpool can and does re-use per-thread-and-process mapnik Map objects. To keep the memory requirement bounded, this patch also adds a simple garbage collector. It kicks in when there are far more cached Map objects than active threads.
This avoids delays after periods of inactivity due to reaped threads by preparing Map objects for the least recently used mapfile.
This reverts commit e58fdeb.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.