-
Notifications
You must be signed in to change notification settings - Fork 23
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
support reading/loading raw cloud-optimized geotiff #338
Comments
Depends on openlayers/openlayers#11810 ... or openlayers/openlayers#12008 ;-) |
hah, i thought more about cramming smth on top of the existing viewer modes, but right if that comes for free with an openlayers/leaflet upgrade.. |
cf mviewer/mviewer#698 too :) |
as seen in the mviewer issue above and the progress done by @Gaetanbrl to support COG in mviewer, this requires OL >= 7.1.1. |
@tdipisa: what is expected for the estimation of this issue is:
we have plenty of EPSG:2154 COGs available online for testing |
Dear @landryb the aim of the POC I've suggested for this is to well evaluate the integration of COG in MapStore. Since it is a completely new element, under a certain extent of course, it is not easy to foresee all the possibilities until we start the implementation. That's the same approach we have followed for the 3D Tiles implementation that allowed us to integrate it and better understand possibilities to evolve it in a good way: what we have done effectively in 2022.
yes
yes
yes
yes
To evaluate, but I think it should be possible
I cannot guarantee for further things for now but, given our estimates, we will evaluate what can be included within them and we will do that if possible
That's really useful of course. Thank you for sharing when needed. |
@catmorales @landryb after our yesterday's exchanges I can confirm that these two points are included in our estimates:
|
thanks for the confirmation - perfect ! |
i thought about the projection bit, and the necessity or not to add a field in the UI so that the user/data admin can specify the projection of the remote COG. A proper COG file is expected to have a valid projection in the tiff header, which should be considered the ultimate source of truth for the projection. I don't think there are many COG files without projection around (or they might be considered 'invalid COG files'?), for example QGIS only allows to load a COG from an URL without specifying its projection in the raster add from url dialog: so Mapstore should read the projection from the COG, and take it into account to eventually reproject raster data on the fly if it's different from the viewer projection. Pretty sure that's what https://geotiffjs.github.io/cog-explorer/ does. |
i guess made possible with the openlayers update in geosolutions-it/MapStore2#9293 |
MS issue ref: |
for ms2-georchestra users: to test this feature, don't forget to add some entries to |
@landryb Of course and btw, these
|
oh you're right, hadnt checked that those projections definitions were present in ms2-geor's localConfig.json (https://github.com/georchestra/mapstore2-georchestra/blob/master/configs/localConfig.json#L53), and since i was doing my testing in bare ms2 that's why i had missing projection defs. so it should be fine by default in ms2-geor :) |
@tdipisa i'm not sure if feedback should be in this issue (since its closed) or a different one, but:
i've also noted that the catalog add COG dialog refuses an url that contains percent-encoded chars. the above url will fail, but works if i replace |
I think I solved my problem :
|
i think i've used BLOCKSIZE=512 for all my COGs as iirc that's what is advised |
Should be more in geosolutions-it/Mapstore2 project but oh well..
nowadays, the new trend to serve background layers is cogeo (cf https://www.cogeo.org/) - using https://geotiffjs.github.io/ it should be doable in Mapstore2 to have a new layer type allowing to load a remote cloud-optimized geotiff directly in the browser (an example at https://geotiffjs.github.io/cog-explorer/), only giving an url (and eventual credentials ?) - no more WMS, no more caching.. a dream come true ?
The text was updated successfully, but these errors were encountered: