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

Dynamically changing debugFreezeFrame can cause issues on large tilesets #12233

Closed
BlockCnFuture opened this issue Oct 6, 2024 · 1 comment
Closed

Comments

@BlockCnFuture
Copy link

What happened?

image
image
I wrote a sandcastle to illustrate the logic of what I'm doing, but I don't have access to a large model so this sandcastle may not reproduce the problem, it's just to illustrate what I'm doing

Reproduction steps

Since large models will reload distant tiles when the camera moves, this will cause serious FPS drops and cause the model to move lags, so I enable frame froze when the mouse is pressed and disable frame froze when the mouse is released. However, this error appeared when I moved the model repeatedly many times.

Sandcastle example

https://sandcastle.cesium.com/#c=xVTbitswEP0VEfoggyunZJdCmoQuubCBtLs06ebF0JXlSSIqS0aSHZIl/175lngvfSgUVi/2jGbOnDnSiClpLMo57EGjIZKwR2MwPEvIQ+nDYYeV9lhJS7kEHXZ89BRK5JYFrZ2v36SsKptstErWSou4dmDPD+XJ+xLKUFp9aNJZWdtyAQasK073lNsGq/r0Jqtqu8ScK3ljnDGP8efr3lXvTKRYIGkkYAJRtl1zDRtNE+gjqzPwq6CSQPFTdUsMAwkk1TzhludgCI1jXLNpQmuTxAXsWAml+RFKSo5vgV321AI9KpWsFL7QqhH8i6el8S3QmMvtPbds94PKLeAu6froY5dc++fakcpkEbVMd6CBaJeSGRSgK9L1aljvTKPSdEdlLF4e6JJpALlMKYNpDtLeVkH4mRyMypyaC16NRArVZZrZG2a5khh7aDhqq/9MqJkrdIRZcQIXmcoj8P9GZnVIgSyms9Wvyd36+3+tv6HC/AuBn/fv1P63+WSymL6rADWFRoITYtRdTYTdHCvttQdXCSBCbfHjtNhCQpU3uaHRRx+eypzTY4F06vidgbEHAaOG8VeepEpblGmBCQksJKmgFkwQZey364MZ0wzhIGinDmKeIx4P33iYEBPUGLezyYRYukkNO6NB4OJfpdZ073LQgh6KsN2n0aJyEkIGgTPfzrRKiYjqF8h/AA

Environment

Browser: electron 25.2.0
CesiumJS Version: 1.120
Operating System: win11

@ggetz
Copy link
Contributor

ggetz commented Oct 11, 2024

Hi @BlockCnFuture, thanks for the report!

This property is for debugging only; it is not optimized for production use.

Strictly speaking, yes, this is a bug. But since it appears to be a corner case of a debugging feature, we are unlikely to provide a fix ourselves so I'm going to close the issue.

However, if you are interested in contributing a fix, we would be happy to review a PR!

@ggetz ggetz closed this as completed Oct 11, 2024
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