Skip to content

Conversation

@azeey
Copy link
Contributor

@azeey azeey commented Aug 12, 2025

➡️ Forward port

Port gz-sim8 ➡️ gz-sim9

Branch comparison: gz-sim9...gz-sim8

Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

arjo129 and others added 30 commits October 8, 2021 14:43
Signed-off-by: Arjo Chakravarty <[email protected]>
Signed-off-by: Arjo Chakravarty <[email protected]>
Signed-off-by: Arjo Chakravarty <[email protected]>
…cs/ign-gazebo into arjo/fix/detachable_joint
Signed-off-by: Michael Carroll <[email protected]>
CMake checks if rendering is available to enable some tests, so the
rendering environment needs to be set before CMake runs.

---------

Signed-off-by: Addisu Z. Taddese <[email protected]>
Signed-off-by: Addisu Z. Taddese <[email protected]>
Merge ign-gazebo3 ➡️  ign-gazebo6
Signed-off-by: Michael Carroll <[email protected]>
Signed-off-by: Michael Carroll <[email protected]>
* Disable failing testFixture_TEST for MacOS

Signed-off-by: Crola1702 <[email protected]>

* Add comment with issue

Signed-off-by: Crola1702 <[email protected]>

* Update issue comment

Signed-off-by: Crola1702 <[email protected]>

---------

Signed-off-by: Crola1702 <[email protected]>
n particular if a user despawns an entity, the associated plugin gets removed. This should prevent issues like gazebosim#2165. TBH I'm not sure if this is the right way forward as a system should technically be able to access any entity in a traditional ECS.

The PR has now been reworked and greatly simplified. All we do is stop all worker threads if an entity is removed and then recreate remaining threads.

Signed-off-by: Arjo Chakravarty <[email protected]>
Signed-off-by: Ian Chen <[email protected]>
Signed-off-by: Arjo Chakravarty <[email protected]>
Signed-off-by: Arjo Chakravarty <[email protected]>
- Add better logging
- Filter by models only
- Check child link immediately

Signed-off-by: Arjo Chakravarty <[email protected]>
Signed-off-by: Arjo Chakravarty <[email protected]>
Signed-off-by: Arjo Chakravarty <[email protected]>
Signed-off-by: Arjo Chakravarty <[email protected]>
Signed-off-by: Arjo Chakravarty <[email protected]>
…joint

Fix an issue where an incorrect child model is selected in the DetachableJoint system if multiple models with the given `child_model` name are present. The updated logic resolves this scenario by selecting the first model that 
- is a child of the parent model and 
- also has a link with the given `child_link` name.

---------

Signed-off-by: Arjo Chakravarty <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Co-authored-by: Louise Poubel <[email protected]>
mergify bot and others added 15 commits January 27, 2025 01:01
gazebosim#2732) (gazebosim#2735)

Signed-off-by: Josh Newans <[email protected]>
(cherry picked from commit 06e16bc)

Co-authored-by: Josh Newans <[email protected]>
…gazebosim#2742)

- Reduces serialization warnings on `sdf::Model` so it only occurs when
  the `relative_to` attribute is set. This was the intended behavior
  originally, but there was a bug in the code.
- Eliminate `sdf::World` warnings by specializing the `DefaultSerializer` for `sdf::World`.

---------

Signed-off-by: Addisu Z. Taddese <[email protected]>
(cherry picked from commit 1739bdb)
Signed-off-by: Addisu Z. Taddese <[email protected]>
Signed-off-by: Erling Rennemo Jellum

(cherry picked from commit 13bf5be)

Signed-off-by: Erling Rennemo Jellum
Signed-off-by: erlingrj <[email protected]>
Signed-off-by: erling <[email protected]>
Signed-off-by: Addisu Z. Taddese <[email protected]>
Co-authored-by: erling <[email protected]>
Co-authored-by: Addisu Z. Taddese <[email protected]>
Co-authored-by: Ian Chen <[email protected]>
Signed-off-by: Addisu Z. Taddese <[email protected]>
(cherry picked from commit 6a2df65)
Signed-off-by: Addisu Z. Taddese <[email protected]>
(cherry picked from commit 6a2df65)
Loading the logging playback plugin before `SceneBroadcaster`
prevents playback from being displayed on the GUI.

---------

Signed-off-by: Addisu Z. Taddese <[email protected]>
Co-authored-by: Arjo Chakravarty <[email protected]>
(cherry picked from commit ca40c1d)

# Conflicts:
#	src/SimulationRunner.cc
Signed-off-by: Arjo Chakravarty <[email protected]>
Signed-off-by: Arjo Chakravarty <[email protected]>
Signed-off-by: Arjo Chakravarty <[email protected]>
In `SimulationRunner`, we initialize a worker pool but we never actually
use it so it's pure overhead. In `ServerPrivate` we only use the
worker pool if there are multiple simulation runners, so we can optimize
for the most common use case of one runner.

Signed-off-by: Addisu Z. Taddese <[email protected]>
(cherry picked from commit 5ce62a8)
Signed-off-by: Addisu Z. Taddese <[email protected]>
@azeey azeey requested a review from arjo129 as a code owner August 12, 2025 15:02
@github-actions github-actions bot added the 🏛️ ionic Gazebo Ionic label Aug 12, 2025
Signed-off-by: Addisu Z. Taddese <[email protected]>
@azeey azeey merged commit 502e99d into gazebosim:gz-sim9 Aug 13, 2025
13 of 14 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in Core development Aug 13, 2025
@azeey azeey deleted the 8_to_9 branch August 13, 2025 20:03
@azeey azeey added this to the Jetty Release milestone Aug 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏛️ ionic Gazebo Ionic

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.