Thanks for the pull request @lilleyse!

• ✔️ Signed CLA found.

Reviewers, don't forget to make sure that:

• Cesium Viewer works.
• Works in 2D/CV.
• Works (or fails gracefully) in IE11.

Thanks @lilleyse ! This is the right fix in my opinion, since 3D Tilesets are assumed to be stationary by default, and this is an additional optimization we make in the common case, that we can just turn off for this uncommon case. I can confirm this fixes the issue brought up in #8580 (comment).

I was going to ask if we should consider making a getter for cullRequestsWhileMoving since tileset.cullRequestsWhileMoving is no longer the "true" value, but I think it's an internal detail that this is disabled when the tileset is moving. So if you set this value to true, it should be true when you query it, even when the tileset is moving, since we document that this optimization only applies for stationary tilesets.

The only way I can think to break this is if the tileset happens to be moving by modifying tileset.root.transform instead of tileset.modelMatrix (Sandcastle example). Although I would say this isn't worth the extra check to account for.

Thoughts?

The only way I can think to break this is if the tileset happens to be moving by modifying tileset.root.transform instead of tileset.modelMatrix (Sandcastle example). Although I would say this isn't worth the extra check to account for.

Ah yes! I thought of that case as well but forgot to mention it here. I looked at the code in #8580 to make sure it was targeting modelMatrix instead of the root transform. I also agree it's not worth accounting for since it's not really the "sanctioned" way of moving a tileset, but if it does happen it will be an easy fix.