Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Port Server changes from gz-sim8 to gz-sim9 (#2638 and #2754) #2760

Merged
merged 2 commits into from
Feb 21, 2025

Conversation

iche033
Copy link
Contributor

@iche033 iche033 commented Feb 5, 2025

➡️ Forward port

Port #2638 and #2754 to gz-sim9, done by cherry-picking d94c11e and 78954a5

Branch comparison: gz-sim9...breakout_server_9

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

@iche033 iche033 requested a review from mjcarroll as a code owner February 5, 2025 23:25
@github-actions github-actions bot added the 🏛️ ionic Gazebo Ionic label Feb 5, 2025
@azeey azeey requested a review from nkoenig February 5, 2025 23:33
nkoenig and others added 2 commits February 21, 2025 19:36
* Copy the code to the new function - won't compile

* Fix the build

* Replace static default world with sdf default world

* Added string and utility headers

* test moving header file

Signed-off-by: Nate Koenig <natekoenig@gmail.com>

* Move header file back

Signed-off-by: Nate Koenig <natekoenig@gmail.com>

* Move meshinertiacalculator header

Signed-off-by: Nate Koenig <natekoenig@gmail.com>

* Testing remove of literals

Signed-off-by: Nate Koenig <natekoenig@gmail.com>

* Undo tests, and just move the MeshInertiaCalculator.hh to top of file

Signed-off-by: Nate Koenig <natekoenig@gmail.com>

---------

Signed-off-by: Nate Koenig <natekoenig@gmail.com>
@iche033 iche033 merged commit daa2460 into gz-sim9 Feb 21, 2025
12 checks passed
@iche033 iche033 deleted the breakout_server_9 branch February 21, 2025 22:44
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.

3 participants