-
Notifications
You must be signed in to change notification settings - Fork 208
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
GPU Accelerated Point In Polygon Test #1360
Conversation
modules/experimental/src/gpgpu/point-in-polygon/gpu-point-in-polygon.js
Outdated
Show resolved
Hide resolved
const curCount = curVertices.length / SIZE; | ||
const curIds = new Float32Array(curCount).fill(polygonId); | ||
vertices.push(...curVertices); | ||
ids.push(...curIds); |
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.
How big can these arrays get? Would recommend pre-allocating the arrays to avoid re-allocating and they dynamically extend.
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.
We can add array manager support in the future PRs, that can be independent of this feature and used across all luma modules.
For #1356
Background
GPUPointInPolygon
to perform point-in-polygon test.TextureTransform
to be created with source asBuffers
andTexture
as target. (Shouldn't impact any existing use cases)polygon.js
) from deck.gl, we should move this tomath.gl
eventually to avoid duplication.Change List