Add Dataframe message

[caguero]

🎉 New feature

Required by

• Add comms infrastructure gz-sim#1416

Summary

This is a message to mimic a data frame used to send and receive data over the network.

Test it

You can test it via gazebosim/gz-sim#1416

Note: This is ready to review but do not merge it yet please. I'm working on a Gazebo system that will use this message and want to confirm that no extra fields are needed.

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 #238 (a2119ff) into ign-msgs8 (962ae76) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           ign-msgs8     #238   +/-   ##
==========================================
  Coverage      85.38%   85.38%           
==========================================
  Files              9        9           
  Lines            951      951           
==========================================
  Hits             812      812           
  Misses           139      139           

---

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 962ae76...a2119ff. Read the comment docs.

[arjo129]

While a header with the correct time stamp and the source and destination addresses are sufficient for the MBZIRC use case, we should also log the position of where the message is starting from this is useful for the MBARI use case. The reason is that the message take in the order of a few seconds to propagate hence it would help if the original position were logged as the vehicle may have moved in this time frame.

[caguero]

Does it make sense to store that information in the specific comms model during the step that's supposed to process this message for the first time? In addition, the client might not know its own location.

[arjo129]

That's one possibility, but it would lead to very complicated code. We probably can revisit this afterwards.

[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-04-13-fortress-edifice/1367/1