-
Notifications
You must be signed in to change notification settings - Fork 327
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
Client side raster reprojection #581
base: master
Are you sure you want to change the base?
Conversation
There is an issue with this branch and the ImageWMS example. |
We were interested something similar, not for raster reprojection itself but just getting Cesium to use the OL tile load stack. That way, implementing a custom tile layer (such as GeoPackage) only needs to be done in OpenLayers and Cesium will run through the same stack. Our PR for the work on that is here, with the relevant
Additionally, I went ahead and made a TilingScheme implementation that uses the original OpenLayers Note that the custom |
Thanks @wallw-bits for the infos. It is pretty cool! |
Hi, I am trying to rebase this branch onto ol-cesium v2 (currently working on the branch https://github.com/samuel-girard/ol-cesium/tree/olcs2-raster-reprojection).
I get in fact the same results than I faced last time (screenshots here: #37 (comment)) It turns out these are the examples using an image source ( |
I completed this work for OpenSphere in these two files:
Those classes require Cesium >= 1.46. You can also check out my last comment above which has the details. |
Hi @wallw-bits So far, I have no idea why I get these problems, but I will keep investigating. |
You still have assumptions of a zero-level zoom in the However, neither of those are really your problem in those examples. The changes to OpenSphere were to support tile layers. Both of the examples you are referencing use image layers, which do not use |
Thanks @wallw-bits for your feedback.
The part you are referring to is about The problem I am facing is about |
Restore #562.