-
Notifications
You must be signed in to change notification settings - Fork 289
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
Enable use of ign gazebo -s on Windows (take two) #1764
Enable use of ign gazebo -s on Windows (take two) #1764
Conversation
Signed-off-by: Silvio <silvio@traversaro.it>
Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com> Signed-off-by: Silvio <silvio@traversaro.it>
Signed-off-by: Silvio <silvio@traversaro.it>
This works for me, thanks @traversaro |
The failing tests look unrelated. I'll wait until CI finishes just to be sure. |
I am not completly sure the CI failures are unrelated, as for example |
* Add topic parameter to thrust plugin (#1681) * Add topic parameter. Signed-off-by: Carlos Agüero <caguero@openrobotics.org> * 🎈 6.12.0: bumped minor and updated changelog (#1682) * 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> * Fix reference link in ackermann steering (#1683) Signed-off-by: Kenji Brameld <kenjibrameld@gmail.com> * Fix installation instructions on Ubuntu 22.04 (#1686) Signed-off-by: Silvio Traversaro <silvio@traversaro.it> * Add a service to trigger functionality (#1611) * 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> * Fix loading render engine plugins in GUI (#1694) Signed-off-by: Ian Chen <ichen@osrfoundation.org> * Enable inherited model topic name. (#1689) 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 ResourceSpawner example file (#1701) 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> * Update triggered_publisher.sdf (#1737) found a silly typo that was pushed back in PR (#1611) * Adds sky cubemap URI to the sky.proto's header (#1739) * 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> * Return absolute path when finding a resource (#1741) * 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> * Restore Add System GUI plugin (#1685) * cherry pick aef3020 Signed-off-by: Ian Chen <ichen@osrfoundation.org> * Adding thrust coefficient calculation (#1652) * 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> * Enable/Disable individual hydrodynamic components. (#1692) 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 (#1754) * Fortress: Removed warnings * Removed unused speedlimit file (#1761) Signed-off-by: ahcorde <ahcorde@gmail.com> * Enable use of ign gazebo -s on Windows (take two) (#1764) * 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> * Script and tutorial for generating procedural datasets with Blender (#1412) Signed-off-by: Andrej Orsula <orsula.andrej@gmail.com> * Fix scene_broadcaster_system test (#1766) Signed-off-by: Nate Koenig <nate@openrobotics.org> Signed-off-by: Nate Koenig <nate@openrobotics.org> Co-authored-by: Nate Koenig <nate@openrobotics.org> * Lint Signed-off-by: Michael Carroll <michael@openrobotics.org> * Clean build and update test Signed-off-by: Nate Koenig <nate@openrobotics.org> Signed-off-by: Carlos Agüero <caguero@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> Signed-off-by: Liam Han <liam@openrobotics.org> Signed-off-by: Mabel Zhang <mabel@openrobotics.org> Signed-off-by: Ian Chen <ichen@osrfoundation.org> Signed-off-by: Benjamin Perseghetti <bperseghetti@rudislabs.com> Signed-off-by: Jose Luis Rivero <jrivero@osrfoundation.org> Signed-off-by: Nate Koenig <nate@openrobotics.org> Signed-off-by: Marco A. Gutierrez <marco@openrobotics.org> Signed-off-by: ahcorde <ahcorde@gmail.com> Signed-off-by: Silvio <silvio@traversaro.it> Signed-off-by: Andrej Orsula <orsula.andrej@gmail.com> Signed-off-by: Michael Carroll <michael@openrobotics.org> Co-authored-by: Carlos Agüero <caguero@openrobotics.org> Co-authored-by: Dharini Dutia <dharini@openrobotics.org> Co-authored-by: Kenji Brameld <kenjibrameld@gmail.com> Co-authored-by: Silvio Traversaro <silvio@traversaro.it> Co-authored-by: Liam Han <liam@openrobotics.org> Co-authored-by: Mabel Zhang <mabel@openrobotics.org> Co-authored-by: Ian Chen <ichen@osrfoundation.org> Co-authored-by: Benjamin Perseghetti <bperseghetti@rudislabs.com> Co-authored-by: Jose Luis Rivero <jrivero@osrfoundation.org> Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com> Co-authored-by: Nate Koenig <nate@openrobotics.org> Co-authored-by: Marco A. Gutierrez <marcogg@marcogg.com> Co-authored-by: Arjo Chakravarty <arjo@openrobotics.org> Co-authored-by: Andrej Orsula <orsula.andrej@gmail.com> Co-authored-by: Michael Carroll <michael@openrobotics.org>
New version of #1574 after the DCO errors.
🎉 New feature
This PR enable the use of
ign gazebo -s
on Windows. Similarly to what is done in macOS, asign gazebo
andign gazebo -g
are not supported, those options are explicitly disabled to give a nice error to the users.Summary
Summary of changes:
src/cmd
directory also on Windowsign.hh
andModelCommandAPI.hh
cmdgazebo.rb.in
to load the gazebo pluginUNIT_ign_TEST
to work correctly on Windowsign-tools
when using multiple config CMake generators, and consequently disableUNIT_ign_TEST and
UNIT_ModelCommandAPI_TEST` when using multiple config CMake generators.ign gazebo -s
is working, explicitly disableign gazebo -g
andign gazebo
as done on macOS .Test it
Note, the following instructions are not complete as a version of libignition-common4 with gazebosim/gz-common#389 needs to be released before the tests work again.
Personally, I tested it with conda-forge provided depedencies. I created an environment with:
Then, I compiled
ign-gazebo
with:After that, I was able to run correctly the server:
To verify the server was running correctly, I run in another terminal:
Checklist
codecheck
passed (See contributing)Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining
Signed-off-by
messages.