gazebosim / docs

High-level Gazebo documentation that gets published to https://gazebosim.org/docs/
https://gazebosim.org/docs
48 stars 117 forks source link

Adding additional docs on sensors and ROS 2 usage #433

Closed SteveMacenski closed 6 months ago

SteveMacenski commented 7 months ago

Summary

Adds some important documentation context and linking up of key information in tutorials. I wasn't sure how you handle version control since all the version are in the same branch, so I made the changes across the 3 current and future versions (and made sure for Fortress to use IGN instead of GZ in writing). If that's all automatically backported, let me know and I'll avoid that in the future.

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.

mabelzhang commented 7 months ago

For clarification - This is a feature / bug fix right? Could you please remove the "Forward port" and "Release" sections in the PR description?

I wasn't sure how you handle version control since all the version are in the same branch, so I made the changes across the 3 current and future versions

Yeah what you did is correct.

mabelzhang commented 7 months ago

Maybe @quarkytale is the most familiar with other relevant tutorials to review parts of this one? No obligations.

SteveMacenski commented 6 months ago

Any blockers to having this merged now? This feels like something that can go in easily once the instructions are validated over each distribution (thanks Mabel!)

SteveMacenski commented 6 months ago

@mabelzhang when does this go online?

azeey commented 6 months ago

I just ran the deployment. Should be up pretty soon.