Skip to content

Conversation

horverno
Copy link
Contributor

This example shows how to convert a 2D range measurement to a grid map. This is the basis of creating occupancy grid map eg for SLAM.

@coveralls
Copy link

coveralls commented May 31, 2019

Pull Request Test Coverage Report for Build 869

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 47 unchanged lines in 10 files lost coverage.
  • Overall coverage decreased (-0.02%) to 90.421%

Files with Coverage Reduction New Missed Lines %
PathPlanning/LQRRRTStar/lqr_rrt_star.py 1 76.09%
PathPlanning/ClosedLoopRRTStar/pure_pursuit.py 1 96.09%
PathPlanning/ClosedLoopRRTStar/closed_loop_rrt_star_car.py 3 80.31%
PathPlanning/ReedsSheppPath/reeds_shepp_path_planning.py 4 90.41%
ArmNavigation/n_joint_arm_to_point_control/n_joint_arm_to_point_control.py 4 93.59%
PathPlanning/BatchInformedRRTStar/batch_informed_rrtstar.py 5 81.5%
PathPlanning/AStar/a_star.py 5 96.32%
Localization/unscented_kalman_filter/unscented_kalman_filter.py 6 92.19%
PathPlanning/Dijkstra/dijkstra.py 7 94.85%
PathPlanning/DynamicWindowApproach/dynamic_window_approach.py 11 88.7%
Totals Coverage Status
Change from base Build 855: -0.02%
Covered Lines: 6608
Relevant Lines: 7308

💛 - Coveralls

@AtsushiSakai
Copy link
Owner

This pull request introduces 3 alerts when merging affcc3f into 9d1a05a - view on LGTM.com

new alerts:

  • 2 for Unused local variable
  • 1 for Unused import

Comment posted by LGTM.com

@AtsushiSakai
Copy link
Owner

This pull request introduces 1 alert when merging 5d6927a into 9d1a05a - view on LGTM.com

new alerts:

  • 1 for Unused local variable

Comment posted by LGTM.com

@AtsushiSakai
Copy link
Owner

@horverno Thank you so much for your PR. It is interesting. However, opencv dependency is not acceptable for me. Could you remove it?

@horverno
Copy link
Contributor Author

horverno commented Jun 1, 2019

@AtsushiSakai, thanks, will check out if i can get rid of opencv

@kyberszittya
Copy link
Contributor

kyberszittya commented Jun 3, 2019

Only FloodFill was used from OpenCV, I've got rid of it.

@AtsushiSakai AtsushiSakai merged commit 7c77e01 into AtsushiSakai:master Jun 3, 2019
@AtsushiSakai
Copy link
Owner

@horverno @kyberszittya Thank you so much for you guys contribution!!.

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.

4 participants