-
Notifications
You must be signed in to change notification settings - Fork 263
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
Forward port 7 to main #2023
Forward port 7 to main #2023
Conversation
Signed-off-by: Louise Poubel <louise@openrobotics.org>
Signed-off-by: methylDragon <methylDragon@gmail.com>
Signed-off-by: Louise Poubel <louise@openrobotics.org> Signed-off-by: Louise Poubel <louise@openrobotics.org>
# 🦟 Bug fix Supercedes #1697. Note: When forward porting we will have to update the hashmaps to erase the new hashmaps created. Fixes #<NUMBER> ## Summary Found that when actors are De-spawned the actor visuals are not destroyed. This commit addresses this bug by adding the missing remove logic in RenderUtils. ## Before ![bug](https://user-images.githubusercontent.com/542272/189558600-196d98c5-1dcf-4d6c-93d6-7493df38c0e4.gif) ## After ![no_bug](https://user-images.githubusercontent.com/542272/189558924-3f2e3c5d-68f3-4d80-aee4-3dc3ce6742a1.gif) ## Notes: Theres a lot of hashmaps being populated in RenderUtils whenever a new actor is spawned. I hope I've caught them all. Also while I need these working in garden (as all the projects Im working on use garden), should I backport these changes? Signed-off-by: Arjo Chakravarty <arjo@openrobotics.org>
Signed-off-by: Nate Koenig <nate@openrobotics.org>
…bosim Update examples to use gazebosim.org
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
* Convert ignitionrobotics to gaazebosim in tests directory Signed-off-by: Nate Koenig <nate@openrobotics.org> * fix gz-gazebo 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> Signed-off-by: Nate Koenig <nate@openrobotics.org> Co-authored-by: Nate Koenig <nate@openrobotics.org>
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> Signed-off-by: Nate Koenig <nate@openrobotics.org> Co-authored-by: Nate Koenig <nate@openrobotics.org>
Signed-off-by: youhy <haoyuan2019@outlook.com> Signed-off-by: youhy <haoyuan2019@outlook.com>
Signed-off-by: Jenn Nguyen <jenn@openrobotics.org>
Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
In aboutDialog (line 178), the binding loop was created because the height of aboutDialog wasn't set. It was dependent on the height of its child (aboutMessage). But aboutMessage had anchor.fill : parent set which binds its height to the height of its parent. Similar reasoning applies to fileSaveFailure. Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
Signed-off-by: Jenn Nguyen <jenn@openrobotics.org>
Since the ign->gz rename, two canonical Fuel servers ( fuel.ignitionrobotics.org and fuel.gazebosim.org) were being queries by ResourceSpawner. This PR removes the duplication. Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Added PI based example controller for speed, pitch and yaw for LRAUVs. Signed-off-by: Aditya <aditya050995@gmail.com>
Often when sensing we may be interested in vector quantities. This PR extends the environmental_sensor to support vector quantities. Some additional work was required handle transformations. In particular if the data is something like Ocean Currents or Wind then the data has to be transformed to the robot's local coordinate frame and the robot's velocity also has to be accounted for. Signed-off-by: Arjo Chakravarty <arjo@openrobotics.org>
The dartsim component of gz-physics is currently not built on CI, so these tests should be disabled. Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
) * Adds support for Hydrodynamic Current This PR adds support for looking up hydrodynamic currents from a data file via the new Environment component. This allows users to vary currents throughout the water column. Signed-off-by: Arjo Chakravarty <arjo@openrobotics.org> * Codecheck Signed-off-by: Arjo Chakravarty <arjo@openrobotics.org> * more style Signed-off-by: Arjo Chakravarty <arjo@openrobotics.org> * Address PR Feedback Signed-off-by: Arjo Chakravarty <arjo@openrobotics.org> * Fix a few more outstanding issues. Signed-off-by: Arjo Chakravarty <arjo@openrobotics.org> * Address second pass Signed-off-by: Arjo Chakravarty <arjo@openrobotics.org> * Final touches. Signed-off-by: Arjo Chakravarty <arjo@openrobotics.org> * Retrigger CI Signed-off-by: Arjo Chakravarty <arjo@openrobotics.org> * style Signed-off-by: Arjo Chakravarty <arjo@openrobotics.org> Signed-off-by: Arjo Chakravarty <arjo@openrobotics.org>
Signed-off-by: Marco A. Gutierrez <marco@openrobotics.org>
This adds the ISystemConfigureParameters interface, which allows systems to declare parameters. This uses a transport::parameters::ParametersRegistry and creates services for listing, getting, and setting parameters. Based on gazebosim/gz-transport#305. Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
Signed-off-by: Marco A. Gutierrez <marco@openrobotics.org> Signed-off-by: Marco A. Gutierrez <marco@openrobotics.org>
Fix for handling Ackermann wheel steering angle bug where wheel always points same direction when linVel is zero and abs(angVel) > 0.001. Signed-off-by: Benjamin Perseghetti <bperseghetti@rudislabs.com>
Signed-off-by: Jenn Nguyen <jenn@openrobotics.org>
Forward port 6 to 7
Signed-off-by: Benjamin Perseghetti <bperseghetti@rudislabs.com> Co-authored-by: Addisu Z. Taddese <addisu@openrobotics.org>
Updated SdfEntityCreator to create projector entities Updated rendering component to load and create projector objects in the scene Updated scene broadcaster to publish scene info projectors Added conversion function for serializing / deserializing sdf::Projector to / from msgs::Projector Signed-off-by: Ian Chen <ichen@openrobotics.org>
Signed-off-by: Mabel Zhang <mabel@openrobotics.org>
update pre-commit-hooks Signed-off-by: mosfet80 <realeandrea@yahoo.it>
As of gazebosim/sdformat#1117, SDFormat supports joints under world (//world/joint in xpath). This PR adds support for it in gz-sim. Note that some systems that operate on joints, such as the JointPositionController, require that the system is instantiated inside a model, and therefore, will not work with world joints. It's not clear whether this is a real problem since world joints are most likely to be fixed joints used to attach two models together. Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
…2006) Signed-off-by: Silvio Traversaro <silvio@traversaro.it> * TriggeredPublisher: don't catch FatalException It has been removed from recent versions of protobuf. Signed-off-by: Steve Peters <scpeters@openrobotics.org> Co-authored-by: Steve Peters <scpeters@openrobotics.org>
Signed-off-by: Nate Koenig <natekoenig@gmail.com>
Signed-off-by: Nate Koenig <natekoenig@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a couple of minor comments. Otherwise, LGTM!
@azeey I addressed your feedback, PTAL. |
Codecov Report
@@ Coverage Diff @@
## main #2023 +/- ##
==========================================
+ Coverage 64.48% 64.96% +0.47%
==========================================
Files 341 353 +12
Lines 27143 28576 +1433
==========================================
+ Hits 17502 18563 +1061
- Misses 9641 10013 +372
|
Signed-off-by: Michael Carroll <mjcarroll@intrinsic.ai>
533c1ed
to
9ab227e
Compare
Signed-off-by: Michael Carroll <mjcarroll@intrinsic.ai>
Thanks @mjcarroll |
➡️ Forward port
Port gz-sim7 to main. This one is slightly newer to #2011, and includes some recent changes.
Branch comparison: https://github.com/gazebosim//compare/<TO_BRANCH>...<FROM_BRANCH>
Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)