Add: filter to remove locally maximal points in the z dimension#577
Add: filter to remove locally maximal points in the z dimension#577taketwo merged 1 commit intoPointCloudLibrary:masterfrom
Conversation
There was a problem hiding this comment.
This duplicates (and hides) the same function in FilterIndices. Any reason for having this (and getNegative)?
|
@taketwo I've made the requested changes. Looks like Travis should pass as well. |
There was a problem hiding this comment.
Missing space here. And since you anyway will modify the line, we cloud change int to size_t to avoid compiler warnings.
|
I guess the algorithm may be accelerated quite a bit. Whenever you find a point that is the local maximum, you can safely exclude all of its neighbors from the processing queue because they are guaranteed not to be maximum in their region. That will save a number of radius searches. But that's more of a loud thinking. I think we could merge as is. |
|
It's a good thought and doesn't seem like a lot of work. Since I've got another update to make as it is, I'll go ahead and investigate this. |
|
@taketwo Just pushed the update to this one. Take a look when you get a chance. |
Add: filter to remove locally maximal points in the z dimension
|
Great, thank you! |
2 participants
No description provided.