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

Merge 8 -> main #2378

Merged
merged 8 commits into from
May 2, 2024
Merged

Merge 8 -> main #2378

merged 8 commits into from
May 2, 2024

Conversation

iche033
Copy link
Contributor

@iche033 iche033 commented Apr 19, 2024

➡️ Forward port

Port gz-sim8 to main

Branch comparison: main...gz-sim8

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

azeey and others added 8 commits April 9, 2024 13:53
Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
Signed-off-by: retinfai <aferetipama@gmail.com>
…ata (#2358)

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
# Summary
The examples/worlds directory does not contain any example world for DopplerVelocityLogSystem plugin. This PR adds the following:

Example world based on test/worlds/flat_seabed.sdf but with a SceneBroadcaster.
README.md in test/worlds directory to inform users not to use those world files.
More information can be found in Issue #2344 and PR #2346.

# Test it
This world can be tested (after building and sourcing the workspace) using the following command
```
  gz sim flat_seabed.sdf
```
---------

Signed-off-by: Saurabh Kamat <kamatsaurabh01@gmail.com>
* Do not need distutils on jammy

Signed-off-by: Jose Luis Rivero <jrivero@osrfoundation.org>

---------

Signed-off-by: Jose Luis Rivero <jrivero@osrfoundation.org>

---------

Signed-off-by: Ian Chen <ichen@openrobotics.org>
Signed-off-by: Ian Chen <ichen@openrobotics.org>
@iche033 iche033 requested a review from mjcarroll as a code owner April 19, 2024 18:25
@github-actions github-actions bot added the 🏛️ ionic Gazebo Ionic label Apr 19, 2024
@azeey azeey self-requested a review April 29, 2024 18:25
Copy link

codecov bot commented May 2, 2024

Codecov Report

Attention: Patch coverage is 73.33333% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 65.95%. Comparing base (ae75c25) to head (098085b).
Report is 74 commits behind head on main.

Files Patch % Lines
src/ServerConfig.cc 57.14% 3 Missing ⚠️
src/Server.cc 83.33% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2378      +/-   ##
==========================================
+ Coverage   65.66%   65.95%   +0.28%     
==========================================
  Files         324      327       +3     
  Lines       30938    31319     +381     
==========================================
+ Hits        20315    20655     +340     
- Misses      10623    10664      +41     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@azeey
Copy link
Contributor

azeey commented May 2, 2024

I think the warning on Jammy is an existing issue and should be fixed in a follow up PR. This is good to merge.

@mjcarroll mjcarroll merged commit ad3ef0a into main May 2, 2024
8 of 9 checks passed
@mjcarroll mjcarroll deleted the merge_8_main_20240419 branch May 2, 2024 20:53
@Blast545
Copy link
Contributor

Blast545 commented May 2, 2024

🧑‍🌾 @azeey The warning on jammy should be fixed after #2388 gets forward ported, should I open a PR for that one or wait until there's a new round of forward ports to main?

PTAL #2395

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏛️ ionic Gazebo Ionic
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

8 participants