-
Notifications
You must be signed in to change notification settings - Fork 370
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
WMTS cache capability in WMTSRasterSource #2314
Comments
The caching is currently per figure for interactive speed per this comment: cartopy/lib/cartopy/io/ogc_clients.py Lines 349 to 357 in 5e44939
A local cache would be a great addition, similar to what is available in the cartopy/lib/cartopy/io/img_tiles.py Lines 59 to 69 in 5e44939
For reference, this is the PR that was added in: |
Thank you for this clarification. Happy to close this now, or leave it open if someone wants to take it on as an enhancement. I'd love to make a PR but likely won't be able to in the immediate future. |
I think it makes sense to leave this open as a feature request. It seems like an obvious thing for people to want and @greglucas has now included some guidance for if/when someone has time to look into it. |
I couldn't help myself and threw a proof of concept PR together (#2316). Bad timing as I will be traveling for the next couple of weeks, but the basic functionality is working. |
Description
I asked a relevant question on SO a few months back but raising as an issue here as the current documentation is unclear and/or misleading.
When accessing a WMTS source,
ax.add_wmts()
callsWMTSRasterSource()
. The docs forWMTSRasterSource
state that it "Uses tile caching for fast repeated map retrievals" and a glance at the code indicates cache-related stuff going on.In my experience, however, no caching is performed, and repeated identical calls result in network activity every time. Are the docs referring to some other kind of caching?
Code to reproduce
Full environment definition
Operating system
macOS 13.6.3
Cartopy version
0.22.0
conda list
pip list
The text was updated successfully, but these errors were encountered: