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.

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

Update max_convex_hulls description #1386

Merged
merged 2 commits into from
Mar 26, 2024
Merged

Update max_convex_hulls description #1386

merged 2 commits into from
Mar 26, 2024

Conversation

iche033
Copy link
Contributor

@iche033 iche033 commented Mar 26, 2024

🦟 Bug fix

Summary

Update <max_convex_hulls> description based on the recommendation in gazebosim/gz-physics#606 (review).

This parameter now determines the max number of submeshes that will be in the final decomposed mesh.

Checklist

  • Signed all commits for DCO
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

@iche033 iche033 requested review from azeey and scpeters as code owners March 26, 2024 02:24
@github-actions github-actions bot added the 🏛️ ionic Gazebo Ionic label Mar 26, 2024
Signed-off-by: Ian Chen <[email protected]>
Copy link

codecov bot commented Mar 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.44%. Comparing base (836dd42) to head (ba3072f).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1386   +/-   ##
=======================================
  Coverage   92.44%   92.44%           
=======================================
  Files         134      134           
  Lines       17812    17812           
=======================================
  Hits        16467    16467           
  Misses       1345     1345           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@iche033 iche033 merged commit 67ead53 into main Mar 26, 2024
10 checks passed
@iche033 iche033 deleted the max_convex_hull_desc branch March 26, 2024 21:00
aagrawal05 pushed a commit to aagrawal05/sdformat that referenced this pull request Apr 2, 2024
iche033 added a commit that referenced this pull request Apr 8, 2024
iche033 added a commit that referenced this pull request Apr 10, 2024
aagrawal05 pushed a commit to aagrawal05/sdformat that referenced this pull request Aug 16, 2024
aagrawal05 pushed a commit to aagrawal05/sdformat that referenced this pull request Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏛️ ionic Gazebo Ionic
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants