-
Notifications
You must be signed in to change notification settings - Fork 281
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
Add world force #1279
Add world force #1279
Conversation
Signed-off-by: Carlos Agüero <[email protected]>
Signed-off-by: Carlos Agüero <[email protected]>
Signed-off-by: Carlos Agüero <[email protected]>
Signed-off-by: Carlos Agüero <[email protected]>
Signed-off-by: Carlos Agüero <[email protected]>
Signed-off-by: Carlos Agüero <[email protected]>
Signed-off-by: Carlos Agüero <[email protected]>
Codecov Report
@@ Coverage Diff @@
## ign-gazebo6 #1279 +/- ##
===============================================
+ Coverage 62.22% 62.23% +0.01%
===============================================
Files 278 278
Lines 23122 23131 +9
===============================================
+ Hits 14387 14396 +9
Misses 8735 8735
Continue to review full report at Codecov.
|
Signed-off-by: Ian Chen <[email protected]>
added test in d1676c3. Marking this PR as ready. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The logic looks good to me.
This pull request has been mentioned on Gazebo Community. There might be relevant details there: https://community.gazebosim.org/t/new-ignition-releases-2022-01-24-citadel-edifice-fortress/1241/1 |
🎉 New feature
Summary
This PR extends the
Link
class with the ability to apply a force with an offset.Thanks to @azeey for all the physics tips!
Test it
Checklist
codecheck
passed (See contributing)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.