Fix vtkPCLRansacModel for planes and add proba parameter
This MR:
- fixes vtkPCLRansacModel filter when matching a plane
- exposes the probability and max iterations parameters for the optimization
- removes some typos
@nicolas.cadart @nick.laurenson please take a look
Merge request reports
Activity
- Resolved by Lea Vauchier
It could be a good idea to also add the Number of iterations parameter, and to set both Probability and Number of iterations parameters as advanced.
- Resolved by Lea Vauchier
for the change@nicolas.cadart could this be easily integrated in the PCL Plugin?
@nick.laurenson Do you mean to add this filter in the PCL plugin? It could be easily done. And this would mean removing this filer from native LidarView. It is again the same debate : using adequate libs that offer more features, or being standalone with less dependencies but also less features.
added 1 commit
- 45982cfb - Fix vtkPCLRansacModel for planes and add proba parameter
- Resolved by Lea Vauchier
@nicolas.cadart Here, we already rely on PCL, so this is not standalone. IMO this should eventually go to the PCL plugin.
mentioned in commit 57e6c9fc