Adding angular acceleration to the Link class

[caguero]

Signed-off-by: Carlos Agüero [email protected]

🎉 New feature

Summary

This pull request extends the Link class adding the ability to get the angular acceleration.

Test it

Checklist

• Signed all commits for DCO
• Added tests
• Added example and/or tutorial
• Updated documentation (as needed)
• Updated migration guide (as needed)
• 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.

[codecov]

Codecov Report

Merging #1288 (73afcaf) into ign-gazebo6 (989cd28) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head 73afcaf differs from pull request most recent head 296fd61. Consider uploading reports for the commit 296fd61 to get more accurate results

@@             Coverage Diff              @@
##           ign-gazebo6    #1288   +/-   ##
============================================
  Coverage        62.22%   62.22%           
============================================
  Files              278      278           
  Lines            23122    23126    +4     
============================================
+ Hits             14387    14391    +4     
  Misses            8735     8735           

Impacted Files	Coverage Δ
include/ignition/gazebo/Link.hh	100.00% <ø> (ø)
src/Link.cc	93.75% <100.00%> (+0.20%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 19aba6d...296fd61. Read the comment docs.

[iche033]

added test in 73afcaf. Marking this PR as ready

[chapulina]

Thanks! Can you just update the tutorial here?

https://github.com/ignitionrobotics/ign-gazebo/blob/989cd28896fb3786f1c7947eff10d09c7ef388bf/tutorials/migration_link_api.md#L144

We often forget to

[iche033]

Can you just update the tutorial here?

yep updated. da78b81

[iche033]

The UNIT_SdfGenerator_TEST test failure looks unrelated and ABI report looks good..

[chapulina]

The UNIT_SdfGenerator_TEST test failure looks unrelated

I shoved a fix for that into #1286 , because that test was behaving differently for different platforms. It's showing up now since the ign-common 4.5.0 release.

[osrf-triage]

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