Save geotiff files and metadata to indexeddb#891
Save geotiff files and metadata to indexeddb#891gjmooney wants to merge 1 commit intogeojupyter:mainfrom
Conversation
|
Thanks @gjmooney ! We should discuss this, I have some concerns. We used to save the geotiff content into indexeddb, but it was removed as part of #503. The issue is that OpenLayers supports COG properly, it only downloads what it needs for rendering, but we were downloading the full geotiff content (could be gigs of data!) |
martinRenou
left a comment
There was a problem hiding this comment.
Putting my veto on this for now
|
Integration tests report: appsharing.space |
|
I may be misunderstanding the PR here. In which case are we using this |
If this is the case, let's update the name! For COGs over HTTP, I think the only thing we should be caching is the min and maximum values. Maybe all the metadata. |
db52fec to
c1d4fb9
Compare
Description
Checklist
Resolves #XXX.Failing lint checks can be resolved with:
pre-commit run --all-filesjlpm run lint📚 Documentation preview: https://jupytergis--891.org.readthedocs.build/en/891/
💡 JupyterLite preview: https://jupytergis--891.org.readthedocs.build/en/891/lite