-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
3D Tiles - WebGL stubs #4938
3D Tiles - WebGL stubs #4938
Conversation
CI will always fail until this is fixed, right? I don't think we can merge this until the fix since it users will no longer be able to easily access the deployed branch. Is it a quick fix or should we add it to #3241 and mark it as
Will review as soon as I can, but might have to be end of the week. |
JSHint warnings:
|
Perhaps unrelated, but I get all these failures when running the tests with the real WebGL implementation on my Mac with Intel. Do you want to fix these now or for me to submit an issue?
|
Code changes look good. |
These same test failures are in the 3d-tiles branch so they are not specific to this refactor. |
This test passes for CI
These will be fixed when the texture compression branch is merged in here. I may just cherry-pick the fixed Texture tests so everything passes. |
I don't get those errors, but they may be a similar issue as below. They don't need to be fixed in this PR.
|
Submitted #4944. |
@lilleyse the |
Continuation of #4827 for 3d-tiles
Notes:
scene.pickForSpecs
just to force a pick to happen. It does not return the picked object. This simplifies some picking tests inCesium3DTilesetSpec
.Otherwise the 3d-tiles code runs fine with webgl-stub on/off, and with browser vs command line.