Skip to content

Conversation

Aglargil
Copy link
Contributor

@Aglargil Aglargil commented Feb 5, 2025

Reference issue

#411

What does this implement/fix?

Add Distance Map for path planning

Additional information

CheckList

  • Did you add an unittest for your new example or defect fix?
  • Did you add documents for your new example?
  • All CIs are green? (You can check it after submitting)

@Aglargil Aglargil changed the title feat add DistanceMap feat: add DistanceMap Feb 5, 2025
@Aglargil
Copy link
Contributor Author

Aglargil commented Feb 5, 2025

@AtsushiSakai
I have implemented the distance map as described and have also added test cases and documentation to enhance the module's robustness and usability. Please review the changes at your convenience. If there are any adjustments or further enhancements needed, I am ready to make the necessary updates promptly.

Thank you for your work on this project, and I look forward to your feedback!

Copy link
Owner

@AtsushiSakai AtsushiSakai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @Aglargil !! Very great PR. Overall, LGTM. I have some suggestions and question.

@Aglargil Aglargil requested a review from AtsushiSakai February 5, 2025 12:42
Copy link
Owner

@AtsushiSakai AtsushiSakai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks.

@AtsushiSakai AtsushiSakai merged commit 322fead into AtsushiSakai:master Feb 5, 2025
8 checks passed
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.

2 participants