gazebosim / gz-physics

Abstract physics interface designed to support simulation and rapid development of robot applications.
https://gazebosim.org
Apache License 2.0
65 stars 40 forks source link

Add Cone as a primitive parametric shape. #638

Closed bperseghetti closed 3 months ago

bperseghetti commented 4 months ago

🦟 Bug fix

Summary

This helps add the missing cone geometry for primitive/basic parametric shapes:

conetopple cone

And is also valuable for visualizations of emitters/source that typically have conic-based spread as seen in this acoustic attack on an IMU by showing the affected area:

drone_attack

Associated PRs:

Checklist

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.

bperseghetti commented 4 months ago

I took @mjcarroll advice and used mesh of cone for the collision instead of attempting to add it into dart. Seen with changes here: https://github.com/gazebosim/gz-physics/pull/638/commits/078395dccde1423c1eb08f418514bf6658a1f221

azeey commented 3 months ago

We usually Rebase and merge for backports, but since this has multiple commits, I'll do a squash merge and update the PR title.