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

Scene.PickPosition And Scene.SampleHeight get incorrect coordinates #10721

Closed
627496967 opened this issue Aug 24, 2022 · 3 comments
Closed

Scene.PickPosition And Scene.SampleHeight get incorrect coordinates #10721

627496967 opened this issue Aug 24, 2022 · 3 comments

Comments

@627496967
Copy link

I recently upgraded the Cesium version of my project, use Scene.pickPosition And Scene.sampleHeight functions do not accurately get the position of the 3Dtiles layer And the Model surface.
When I pick a position on the 3DTiles layer and call this method with the screen coordinates obtained from the event, the resulting world coordinates appear to go directly through the 3DTiles layer. The elevation altitude of the final coordinate is always less than or equal to zero. A similar problem occurs when using the sampleHeight function. The final elevation altitude is be 8000+ or less than 0.
This didn't happen when I was using version 1.87, nor did my 3DTiles layer change. This problem occurs frequently, but not for every pick. This reduces the chance of this happening when I'm looking close to the 3DTiles layer. I don't know where the problem is right now.

Cesium Version: 1.94.3 or 1.96.0
browser: Google Chrome 104.0.5112.102

@627496967
Copy link
Author

I found the same problem in Sandcastle. Incorrect height collected when right - click pick.
https://sandcastle.cesium.com/?src=3D%20Tiles%20Interactivity.html

@gaurav8y785675474
Copy link

hello
i am a beginner in open source world
can you please tell me exact location of code[related to this issue] in files
so that i can work on that

@ggetz
Copy link
Contributor

ggetz commented Apr 3, 2023

The issue with pickPosition is documented in #4368. The issue with Scene.SampleHeight is likely related. In some cases, some refactoring of model classes in 1.97 made this existing bug more obvious.

I'm going to close this report in favor of #4368 to keep discussion in one place. Thanks!

@ggetz ggetz closed this as completed Apr 3, 2023
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

No branches or pull requests

3 participants