Skip to content

Conversation

@jennuine
Copy link
Contributor

🎉 New feature

Summary

Tutorial on preparing DEMs and using them in Gazebo.

Test it

Checklist

  • Signed all commits for DCO
  • Added tests
  • Added example and/or tutorial
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • 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
  • Was GenAI used to generate this PR? If so, make sure to add "Generated-by" to your commits. (See this policy for more info.)

Generated-by: Remove this if GenAI was not used.

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by and Generated-by messages.

Signed-off-by: Jenn Nguyen <[email protected]>
@github-actions github-actions bot added the 🪵 jetty Gazebo Jetty label Jul 19, 2025
@azeey azeey added this to the Jetty Release milestone Jul 28, 2025
@azeey azeey removed this from the Jetty Release milestone Aug 26, 2025
@jennuine jennuine changed the title [WIP] Digital Elevation Model (DEM) tutorial Digital Elevation Model (DEM) tutorial Oct 21, 2025
@jennuine jennuine requested a review from iche033 October 21, 2025 08:22
@jennuine jennuine changed the base branch from main to gz-sim10 October 21, 2025 08:23
@jennuine jennuine marked this pull request as ready for review October 21, 2025 08:23
@jennuine jennuine requested a review from arjo129 as a code owner October 21, 2025 08:24
Copy link
Contributor

@iche033 iche033 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice! I did a first pass review and added some comments

The Ubuntu github CI also picked up some minor lint issues.

@iche033
Copy link
Contributor

iche033 commented Oct 23, 2025

I made some changes in bbb4de5:

  • Added tutorial to manifest so it appears in the generated doxygen tutorial main page
  • Resized images (some were quite big when rendered in the html page)
  • Added some more instructions that I had to do when using QGIS
    • btw I don't have the Google Satellite XYZ connection option. I have OpenStreetMap so I added that to the instruction. I'm on v3.34.4 not sure if that matters.
  • Fixed texture path in sdf

@github-project-automation github-project-automation bot moved this from Inbox to In review in Core development Oct 23, 2025
@jennuine
Copy link
Contributor Author

Thanks Ian!!

btw I don't have the Google Satellite XYZ connection option. I have OpenStreetMap so I added that to the instruction. I'm on v3.34.4 not sure if that matters.

Hm, I'm using 3.44.3. I'll see if there were separate instructions for google satellite, if so I'll add it in a follow up PR.

@osrf-jenkins please run tests

@jennuine jennuine enabled auto-merge (squash) October 23, 2025 16:42
@jennuine jennuine merged commit b5c45ba into gz-sim10 Oct 23, 2025
13 of 14 checks passed
@jennuine jennuine deleted the jennuine/dem_tutorial branch October 23, 2025 17:40
@github-project-automation github-project-automation bot moved this from In review to Done in Core development Oct 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🪵 jetty Gazebo Jetty

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants