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

No Clamp To Ground for Google's 3D Tiles #11305

Closed
ryanhlewis opened this issue May 23, 2023 · 2 comments
Closed

No Clamp To Ground for Google's 3D Tiles #11305

ryanhlewis opened this issue May 23, 2023 · 2 comments

Comments

@ryanhlewis
Copy link

Trying to use both markers or billboards with Google's Photorealistic 3D Tiles makes all entities appear under the surface of the tiles.
Adjusting z height doesn't work, while old fixes like CLAMP_TO_GROUND error out because we have the globe disabled.

Using other fixes, like .clampToHeightMostDetailed or .clampToHeight also do not work, and still put the point under the surface.

The only one that seemed to work was using scene.pickPosition(movement.position), which in context, made a marker wherever I clicked from the example code, and this always seemed to be on top of the terrain. However, I can't simulate the same movement.position for all the dynamically generated markers I wish to put on the terrain on load.

Is there some fix to this? I've seen Google's implementation using CesiumJS on their demo page, which seems to have markers on top of the terrain correctly. However, as with all things Google, their code is heavily obfuscated.

@ggetz
Copy link
Contributor

ggetz commented May 26, 2023

Hi @ryanhlewis, thanks for the request!

Google's examples make use of classification, such as in this sandcastle examples.

However, I agree that we're lacking the cohesive ability to clamp to 3D Tiles as terrain, such as the use case with Google's Photorealistic 3D Tiles, throughout all of the API.

Any luck with Scene.sampleHeight? I would suggest using scene.pickPosition or that as a workaround in the meantime.

@ggetz
Copy link
Contributor

ggetz commented May 26, 2023

Actually, I believe this is a duplicate of #7044.

I'm going to close your issue now to keep the discussion in one place. If you have any further input on this, please post it there instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants