Skip to content
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

Update to work with Cesium 1.59 #1

Open
wants to merge 77 commits into
base: master
Choose a base branch
from
Open

Conversation

Flowm
Copy link

@Flowm Flowm commented Jul 4, 2019

The plugin fails in Cesium 1.59 with RuntimeError: Fragment shader failed to compile. Compile log: ERROR: 0:2: 'czm_ellipsoid' : syntax error

czm_ellipsoid and thus czm_ellipsoid.inverseRadii were replaced by constants in CesiumGS/cesium#7944.
Use the new czm_ellipsoidInverseRadii instead to make the plugin work with Cesium 1.59.

@icemagno
Copy link

icemagno commented Aug 2, 2019

I face an already known bug in Chrome here. The original bug is CesiumGS/cesium#7798

@Flowm Flowm requested a review from xagriff January 26, 2020 01:46
@Flowm
Copy link
Author

Flowm commented Jan 26, 2020

@icemagno Bug seems to be resolved in latest Chrome version.
@xagriff Could you have a look and merge?

@icemagno
Copy link

Hi xagriff. Since I can't see any practical use for the sensors I have no using it anymore. I think I could make some kind of calculations like viewshed or detections but as far I can see it is just a dead object.

@Flowm Flowm force-pushed the master branch 2 times, most recently from 6c8758c to 037c681 Compare December 25, 2022 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants