-
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
Add splitDirection property for PointPrimitive #11982
base: main
Are you sure you want to change the base?
Conversation
Added the splitDirection property to PointPrimitive. Updated PointPrimitive Collection and shaders to work with the property.
Added the splitDirection property to PointPrimitive. Updated PointGraphics and PointVisualizer.
Updated Specs PointPrimitive Collection, Point Graphics and PointVisualizer to work with the new splitDirection property
Thank you for the pull request, @YunVlad! ✅ We can confirm we have a CLA on file for you. |
Awesome, thanks for the contribution @YunVlad! We'll review this shortly! |
Fixed formatting in PointGraphics, PointVisualizer, PointPrimitive, PointGraphicsSpec
Resolving conflict
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.
CHANGES.md
Outdated
@@ -1,5 +1,7 @@ | |||
# Change Log | |||
|
|||
- Added SplitDirection property for display PointPrimitive relative to the `Scene.splitPosition`. |
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.
Could you please
- merge in the
main
branch and resolve any conflicts - Add a link to this PR, ie
[11982](https://github.com/CesiumGS/cesium/pull/11982)
@@ -497,6 +500,12 @@ function createVAF(context, numberOfPointPrimitives, buffersUsage) { | |||
componentDatatype: ComponentDatatype.FLOAT, | |||
usage: buffersUsage[DISTANCE_DISPLAY_CONDITION_INDEX], | |||
}, | |||
{ | |||
index: attributeLocations.splitDirection, |
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.
To save on the number of attributes, which is limited, let's pack this single float component along with distanceDisplayConditionAndDisableDepth
as a 4th component.
Description
Add the splitDirection property for PointPrimitive.
Objects for which splitDirection is set as SplitDirection.NONE (by default) will be displayed to the right and left of Scene.splitPosition.
Objects for which splitDirection is set as SplitDirection.LEFT will be displayed to the left of Scene.splitPosition.
Objects for which splitDirection is set as SplitDirection.RIGHT will be displayed to the right of Scene.splitPosition.
This property is necessary for marking points of interest and subsequent analysis of changes over time.
Author checklist
CONTRIBUTORS.md
CHANGES.md
with a short summary of my change