Skip to content
Snippets Groups Projects

Fix vtkPCLRansacModel for planes and add proba parameter

Merged Lea Vauchier requested to merge fix/pclRansacPlane into master
All threads resolved!

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

Edited by Lea Vauchier

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • :thumbsup: 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.

  • Lea Vauchier added 1 commit

    added 1 commit

    • 45982cfb - Fix vtkPCLRansacModel for planes and add proba parameter

    Compare with previous version

  • Lea Vauchier resolved all threads

    resolved all threads

  • Lea Vauchier changed the description

    changed the description

  • Nicolas Cadart resolved all threads

    resolved all threads

  • Nicolas Cadart approved this merge request

    approved this merge request

  • Lea Vauchier resolved all threads

    resolved all threads

  • merged

  • Lea Vauchier mentioned in commit 57e6c9fc

    mentioned in commit 57e6c9fc

  • Please register or sign in to reply
    Loading