Merge forward 6 to 7, 2022-10-21 #1768
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Add topic parameter. Signed-off-by: Carlos Agüero <caguero@openrobotics.org>
* bumped minor and updated changelog Signed-off-by: Dharini Dutia <dharini@openrobotics.org> * fixed changelog as per feedback and updated migration guide Signed-off-by: Dharini Dutia <dharini@openrobotics.org> Signed-off-by: Dharini Dutia <dharini@openrobotics.org>
Signed-off-by: Kenji Brameld <kenjibrameld@gmail.com>
Signed-off-by: Silvio Traversaro <silvio@traversaro.it>
* initial commit to allow plugin to call a service Signed-off-by: Liam Han <liam@openrobotics.org> * adding tutorial and modifying the world sdf Signed-off-by: Liam Han <liam@openrobotics.org> * added test for single input and single service output Signed-off-by: Liam Han <liam@openrobotics.org> * added test for single input and multiple service output Signed-off-by: Liam Han <liam@openrobotics.org> * added test for invalid matching service name => timeout Signed-off-by: Liam Han <liam@openrobotics.org> * modified variables the camelCase Signed-off-by: Liam Han <liam@openrobotics.org> * fixed typo, indentation, grammar, lines that exceeded 80 char Signed-off-by: Liam Han <liam@openrobotics.org> * fixing ubuntu bionic ci issue Signed-off-by: Liam Han <liam@openrobotics.org> * silly syntax mistake on expect_eq Signed-off-by: Liam Han <liam@openrobotics.org> * added three more test cases that addesses incorrect response type, incorrect request type and false result Signed-off-by: Liam Han <liam@openrobotics.org> * WIP: major restructuring and currently working. Requires more cleanup and test Signed-off-by: Liam Han <liam@openrobotics.org> * WIP: fixed preprocessor define bug Signed-off-by: Liam Han <liam@openrobotics.org> * WIP: working but extremely convoluted Signed-off-by: Liam Han <liam@openrobotics.org> * WIP major modification but a lot of errors and tests failed Signed-off-by: Liam Han <liam@openrobotics.org> * stable version: had to revert back to previous work. all tests passed Signed-off-by: Liam Han <liam@openrobotics.org> * modified to use blocking Request method as well as reduce a service worker thread to just one thread with the publisher. all tests passed Signed-off-by: Liam Han <liam@openrobotics.org> * stable version: had to revert back to previous work. all tests passed Signed-off-by: Liam Han <liam@openrobotics.org> * successfully reverted and tested Signed-off-by: Liam Han <liam@openrobotics.org> * fixing PR suggestions Signed-off-by: Liam Han <liam@openrobotics.org> * changed string with 'serv' to 'srv' and included <mutex> to the header Signed-off-by: Liam Han <liam@openrobotics.org> * fixed indentation and removed rep.set_data since it's unused on the client service Signed-off-by: Liam Han <liam@openrobotics.org> * getting rid of the id Signed-off-by: Liam Han <liam@openrobotics.org> * fixed race condition resulting seldom test failure Signed-off-by: Liam Han <liam@openrobotics.org> * changed from triggerSrv to serviceCount. This compensates for the two threads running at different rate Signed-off-by: Liam Han <liam@openrobotics.org> * braces indentation Signed-off-by: Mabel Zhang <mabel@openrobotics.org> * addressing gnu c compiler (gcc) warnings Signed-off-by: Liam Han <liam@openrobotics.org> Signed-off-by: Liam Han <liam@openrobotics.org> Signed-off-by: Mabel Zhang <mabel@openrobotics.org> Co-authored-by: Mabel Zhang <mabel@openrobotics.org>
Signed-off-by: Ian Chen <ichen@osrfoundation.org>
Allows for inheriting model name for robotNamespace when SDF element is not set and provides a debug message showing the topics it subscribes to. Signed-off-by: Benjamin Perseghetti <bperseghetti@rudislabs.com> Co-authored-by: Nate Koenig <nkoenig@users.noreply.github.com>
Add an example file for the ResourceSpawner plugin. I'm using this to link from https://github.com/gazebosim/docs/blob/master/garden/Model_insertion_fuel.md. To improve gazebosim/garden-tutorial-party#1991. Signed-off-by: Jose Luis Rivero <jrivero@osrfoundation.org> Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
found a silly typo that was pushed back in PR (#1611)
* Adds sky cubemap URI to the sky.proto's header Signed-off-by: Nate Koenig <nate@openrobotics.org> * require sdf 12.6 Signed-off-by: Nate Koenig <nate@openrobotics.org> Signed-off-by: Nate Koenig <nate@openrobotics.org> Co-authored-by: Nate Koenig <nate@openrobotics.org>
* Adds sky cubemap URI to the sky.proto's header Signed-off-by: Nate Koenig <nate@openrobotics.org> * Return absolute path when finding a resource Signed-off-by: Nate Koenig <nate@openrobotics.org> Signed-off-by: Nate Koenig <nate@openrobotics.org> Co-authored-by: Nate Koenig <nate@openrobotics.org>
* cherry pick aef3020 Signed-off-by: Ian Chen <ichen@osrfoundation.org>
* adding thrust coefficient calculation Signed-off-by: Marco A. Gutierrez <marco@openrobotics.org> * Update src/systems/thruster/Thruster.cc Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com> Signed-off-by: Marco A. Gutierrez <marco@openrobotics.org> * Update src/systems/thruster/Thruster.cc Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com> Signed-off-by: Marco A. Gutierrez <marco@openrobotics.org> * Update src/systems/thruster/Thruster.cc Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com> Signed-off-by: Marco A. Gutierrez <marco@openrobotics.org> * Update src/systems/thruster/Thruster.cc Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com> Signed-off-by: Marco A. Gutierrez <marco@openrobotics.org> * Update src/systems/thruster/Thruster.cc Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com> Signed-off-by: Marco A. Gutierrez <marco@openrobotics.org> * Update src/systems/thruster/Thruster.cc Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com> Signed-off-by: Marco A. Gutierrez <marco@openrobotics.org> * Update src/systems/thruster/Thruster.cc Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com> Signed-off-by: Marco A. Gutierrez <marco@openrobotics.org> * Update src/systems/thruster/Thruster.cc Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com> Signed-off-by: Marco A. Gutierrez <marco@openrobotics.org> * Update src/systems/thruster/Thruster.hh Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com> Signed-off-by: Marco A. Gutierrez <marco@openrobotics.org> * thrust coefficient test and behavior updates Signed-off-by: Marco A. Gutierrez <marco@openrobotics.org> * making float comparision more robust Signed-off-by: Marco A. Gutierrez <marco@openrobotics.org> * fix float comparision and lint Signed-off-by: Marco A. Gutierrez <marco@openrobotics.org> Signed-off-by: Marco A. Gutierrez <marco@openrobotics.org> Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
This commit enables and disables individual components of the hydrodynamics. This is often useful for debugging odd behaviours of a hydrodynamic model.
* Fortress: Removed warnings
Signed-off-by: ahcorde <ahcorde@gmail.com>
* Enable use of ign gazebo -s on Windows Signed-off-by: Silvio <silvio@traversaro.it> * Update src/CMakeLists.txt Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com> Signed-off-by: Silvio <silvio@traversaro.it> * Fix cmdmodel6.rb and cmdgazebo6.rb contining the same code Signed-off-by: Silvio <silvio@traversaro.it> Signed-off-by: Silvio <silvio@traversaro.it> Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
…1412) Signed-off-by: Andrej Orsula <orsula.andrej@gmail.com>
Signed-off-by: Nate Koenig <nate@openrobotics.org> Signed-off-by: Nate Koenig <nate@openrobotics.org> Co-authored-by: Nate Koenig <nate@openrobotics.org>
Signed-off-by: Nate Koenig <nate@openrobotics.org>
nkoenig
requested review from
mabelzhang,
iche033 and
mjcarroll
as code owners
Codecov Report
@@ Coverage Diff @@
## gz-sim7 #1768 +/- ##
===========================================
- Coverage 64.22% 64.18% -0.05%
===========================================
Files 336 336
Lines 26603 26830 +227
===========================================
+ Hits 17087 17222 +135
- Misses 9516 9608 +92
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
caguero
approved these changes
Oct 26, 2022
|
There are still quite a few test failures, but I'm not sure which are expected at this point. |
|
@osrf-jenkins retest this please |
Signed-off-by: Nate Koenig <nate@openrobotics.org>
mjcarroll
approved these changes
Nov 2, 2022
Signed-off-by: Nate Koenig <nate@openrobotics.org>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
➡️ Forward port
Port
ign-gazebo6togz-sim7Branch comparison: gz-sim7...ign-gazebo6
Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)