-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Publish voxel sandcastles #11909
Publish voxel sandcastles #11909
Conversation
Thank you for the pull request, @jjhembd! ✅ We can confirm we have a CLA on file for you. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As far as I can tell the code looks fine and the sandcastles behave as expected apart from the weird bug above that I haven't been able to reproduce.
Given that I don't know the voxel code as well I'd be more confident with a review from @ggetz too given it's release day. Gabby did you wanna merge the change from your comment above?
Description
This PR moves the two voxel-related Sandcastles from the
/development
subfolder into the main/gallery
. These Sandcastles will now appear on sandcastle.cesium.com for easier testing and linking from documentation.Along the way, some of the Sandcastle code was cleaned up to remove dead code and use more concise syntax.
The relevant Sandcastles are:
Voxels.html
. ThisCustomShader
was updated to render with transparency, to highlight our support of that feature.VoxelPicking.html
. This one renders the voxels fully opaque and demonstrates the lighting capabilities added in #11875.Issue number and link
Resolves #11886.
Testing plan
Load the Voxels and Voxel Picking Sandcastles and verify that the different shapes and input data types are rendered correctly, and that Voxel Picking works, as described in #11828.
Author checklist
CONTRIBUTORS.md
CHANGES.md
with a short summary of my change