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.

Sign up for GitHub

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 CollisionResult::nearest_points #303

Merged
merged 11 commits into from
Jun 13, 2022
Merged

Add CollisionResult::nearest_points #303

merged 11 commits into from
Jun 13, 2022

Commits on Jun 5, 2022

  1. core: account for change of secury_margin account in all collision algos 

    This implements the changed suggested in coal-library#200
    @jcarpent
    jcarpent committed Jun 5, 2022
    Configuration menu
    Copy the full SHA
    8053f58 View commit details
    Browse the repository at this point in the history

  2. core: account for octree change 

    Not sure yet if the change are all correct.
That's why, I decided to split with respect to the content of coal-library#200.
    @jcarpent
    jcarpent committed Jun 5, 2022
    Configuration menu
    Copy the full SHA
    888f051 View commit details
    Browse the repository at this point in the history

  3. test: add test on secury_margin

    @jcarpent
    jcarpent committed Jun 5, 2022
    Configuration menu
    Copy the full SHA
    88c40ea View commit details
    Browse the repository at this point in the history

  4. ci/ros: ctest output on failure

    @jcarpent
    jcarpent committed Jun 5, 2022
    Configuration menu
    Copy the full SHA
    4031c79 View commit details
    Browse the repository at this point in the history

  5. core: fix assert 

    `distance_lower_bound` may be initialized with a lower value than the one contained in sqrDistLowerBound.
    @jcarpent
    jcarpent committed Jun 5, 2022
    Configuration menu
    Copy the full SHA
    b364de4 View commit details
    Browse the repository at this point in the history

  6. core: remove commented line 

    This is indeed expected as the lower bound has been updated in the collision call.
    @jcarpent
    jcarpent committed Jun 5, 2022
    Configuration menu
    Copy the full SHA
    a15d13b View commit details
    Browse the repository at this point in the history

  7. octree: fix bug

    @jcarpent
    jcarpent committed Jun 5, 2022
    Configuration menu
    Copy the full SHA
    41e2dbb View commit details
    Browse the repository at this point in the history

  8. octree: fix missing update of sqrDistLowerBound

    @jcarpent
    jcarpent committed Jun 5, 2022
    Configuration menu
    Copy the full SHA
    74a052a View commit details
    Browse the repository at this point in the history

Commits on Jun 7, 2022

  1. core: restore missing lines

    @jcarpent
    jcarpent committed Jun 7, 2022
    Configuration menu
    Copy the full SHA
    8bc84d2 View commit details
    Browse the repository at this point in the history

Commits on Jun 13, 2022

  1. core: remove useless comment

    @jcarpent
    jcarpent committed Jun 13, 2022
    Configuration menu
    Copy the full SHA
    1b6662d View commit details
    Browse the repository at this point in the history

  2. core: add missing noalias()

    @jcarpent
    jcarpent committed Jun 13, 2022
    Configuration menu
    Copy the full SHA
    526b538 View commit details
    Browse the repository at this point in the history