Skip to content
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 a Radial Density filter #1373

Merged
merged 1 commit into from
Nov 10, 2016
Merged

Add a Radial Density filter #1373

merged 1 commit into from
Nov 10, 2016

Conversation

chambbj
Copy link
Member

@chambbj chambbj commented Nov 10, 2016

This filter creates a new dimension called RadialDensity that records the
density of points within a sphere of specified radius and centered at the
current point.

This filter creates a new dimension called RadialDensity that records the
density of points within a sphere of specified radius and centered at the
current point.
@abellgithub
Copy link
Contributor

Is this really useful on its own? A density for every point? Would expanding GDALWriter to voxels (3D) make more sense?

@hobu
Copy link
Member

hobu commented Nov 10, 2016

Is this really useful on its own? A density for every point? Would expanding GDALWriter to voxels (3D) make more sense?

This is different than voxelizing the data, and you might use this density factor as an input to some other processing technique further on down the line.

@chambbj
Copy link
Member Author

chambbj commented Nov 10, 2016

This is different than voxelizing the data, and you might use this density factor as an input to some other processing technique further on down the line.

What he said.

@abellgithub
Copy link
Contributor

I get that. Seems better as a function instead of a filter.

@hobu
Copy link
Member

hobu commented Nov 10, 2016

Seems better as a function instead of a filter.

I disagree. After computing this with the filter and storing it in output data, a non-PDAL process could certainly take this value and act on its value. In combination with filters.range, this would be useful for throwing out data that is too high or too low of density. If it were buried in a function, we couldn't do those kinds of activities.

@hobu hobu added this to the 1.4 milestone Nov 10, 2016
@hobu hobu merged commit 6f609d7 into master Nov 10, 2016
@hobu hobu deleted the radialdensity branch November 10, 2016 23:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants