Expose the AccessorView
of PropertyAttributePropertyView
#1213
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added in support of CesiumGS/cesium-unreal#1685.
In the case of voxels, we use a megatexture to store each renderable tile's data. If a property matches the type of a texture in Unreal (e.g. float properties), then it's more performant to copy the entire buffer at a time into the texture, instead of converting its values one-by-one.
To do that, we'd need access to the data pointer of the accessor used by the
PropertyAttributePropertyView
, hence theaccessorView
method.Author checklist
CHANGES.md
with a short summary of my change (for user-facing changes).[ ] I have added or updated unit tests to ensure consistent code coverage as necessary.