Add NavSat messages

[chapulina]

🎉 New feature

Part of

• GPS gz-sensors#23

Required by

• GPS sensor in Gazebo gz-sim#519
• GPS sensor in Gazebo gz-sim#519

Related to

• Add GPS / NavSat sensor to sdf9 sdformat#453

Summary

On gazebosim/sdformat#453 we decided to use the more generic "NavSat" name for navigation satellite systems, instead of GPS, which is one specific type. This change should be reflected on the messages too.

This PR is only adding the new NavSat messages, but we should also update the sensors message to use it on Garden.

Here's the ROS 2 NavSatFix message as a reference.

I retargeted this PR from Dome to Fortress because the GPS sensor will need to use spherical coordinates (see gazebosim/gz-sim#981).

Test it

I'm updating the downstream PRs to use this message.

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

🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸

[codecov]

Codecov Report

Merging #206 (ca3cd0f) into ign-msgs8 (f0b2a69) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           ign-msgs8     #206   +/-   ##
==========================================
  Coverage      84.56%   84.56%           
==========================================
  Files              7        7           
  Lines            855      855           
==========================================
  Hits             723      723           
  Misses           132      132           

---

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 f0b2a69...ca3cd0f. Read the comment docs.

[caguero]

Looks good to me with just one minor comment!

[caguero]

Should this be NavSatSensorProtos?

[chapulina]

Good eye! This was actually copy-pasted, there are other messages with the same problem:

$ grep -r "\"Protos\"" proto/ignition/msgs/
proto/ignition/msgs/sensor_noise.proto:option java_outer_classname = "Protos";
proto/ignition/msgs/polylinegeom.proto:option java_outer_classname = "Protos";
proto/ignition/msgs/gps_sensor.proto:option java_outer_classname = "Protos";

I'm not sure what this affects and whether fixing this could break ABI 🤔

[chapulina]

Please don't merge this until the downstream PRs are done; I want to make sure the message has everything we need before releasing.

[chapulina]

A couple of downstream PRs have been approved, so I think this is good to go.

[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-10/1228/1