Forward port 7 to main #2023
Merged
Forward port 7 to main #2023
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
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: methylDragon <[email protected]>
Signed-off-by: Louise Poubel <[email protected]> Signed-off-by: Louise Poubel <[email protected]>
# 🦟 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  ## After  ## 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 <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
…bosim Update examples to use gazebosim.org
Signed-off-by: ahcorde <[email protected]>
Signed-off-by: ahcorde <[email protected]>
* Convert ignitionrobotics to gaazebosim in tests directory Signed-off-by: Nate Koenig <[email protected]> * fix gz-gazebo Signed-off-by: Nate Koenig <[email protected]> Signed-off-by: Nate Koenig <[email protected]> Co-authored-by: Nate Koenig <[email protected]>
Signed-off-by: Nate Koenig <[email protected]> Signed-off-by: Nate Koenig <[email protected]> Co-authored-by: Nate Koenig <[email protected]>
Signed-off-by: Nate Koenig <[email protected]> Signed-off-by: Nate Koenig <[email protected]> Co-authored-by: Nate Koenig <[email protected]>
Signed-off-by: Nate Koenig <[email protected]> Signed-off-by: Nate Koenig <[email protected]> Co-authored-by: Nate Koenig <[email protected]>
Signed-off-by: youhy <[email protected]> Signed-off-by: youhy <[email protected]>
Signed-off-by: Jenn Nguyen <[email protected]>
Signed-off-by: Addisu Z. Taddese <[email protected]>
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 <[email protected]>
Signed-off-by: Jenn Nguyen <[email protected]>
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 <[email protected]>
…ivity (#1799) Closes #1027 Adds spin box that lets users sets the view controller sensitivity. The limits of the spin box are: [0.01, 10.0] with a step size of 0.1. Signed-off-by: Ian Chen <[email protected]>
Signed-off-by: Alejandro Hernández Cordero <[email protected]>
Added PI based example controller for speed, pitch and yaw for LRAUVs. Signed-off-by: Aditya <[email protected]>
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 <[email protected]>
The dartsim component of gz-physics is currently not built on CI, so these tests should be disabled. Signed-off-by: Addisu Z. Taddese <[email protected]>
) * 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 <[email protected]> * Codecheck Signed-off-by: Arjo Chakravarty <[email protected]> * more style Signed-off-by: Arjo Chakravarty <[email protected]> * Address PR Feedback Signed-off-by: Arjo Chakravarty <[email protected]> * Fix a few more outstanding issues. Signed-off-by: Arjo Chakravarty <[email protected]> * Address second pass Signed-off-by: Arjo Chakravarty <[email protected]> * Final touches. Signed-off-by: Arjo Chakravarty <[email protected]> * Retrigger CI Signed-off-by: Arjo Chakravarty <[email protected]> * style Signed-off-by: Arjo Chakravarty <[email protected]> Signed-off-by: Arjo Chakravarty <[email protected]>
Signed-off-by: Marco A. Gutierrez <[email protected]>
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 <[email protected]>
Signed-off-by: Marco A. Gutierrez <[email protected]> Signed-off-by: Marco A. Gutierrez <[email protected]>
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 <[email protected]>
Signed-off-by: Jenn Nguyen <[email protected]>
Forward port 6 to 7
Signed-off-by: Benjamin Perseghetti <[email protected]> Co-authored-by: Addisu Z. Taddese <[email protected]>
Signed-off-by: Michael Carroll <[email protected]>
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 <[email protected]>
Signed-off-by: Mabel Zhang <[email protected]>
update pre-commit-hooks Signed-off-by: mosfet80 <[email protected]>
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 <[email protected]>
…2006) Signed-off-by: Silvio Traversaro <[email protected]> * TriggeredPublisher: don't catch FatalException It has been removed from recent versions of protobuf. Signed-off-by: Steve Peters <[email protected]> Co-authored-by: Steve Peters <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
nkoenig
requested review from
mabelzhang,
iche033,
azeey and
mjcarroll
as code owners
Closed
Signed-off-by: Nate Koenig <[email protected]>
azeey
reviewed
Jul 12, 2023
|
@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 <[email protected]>
mjcarroll
force-pushed
the
nkoenig/7-to-main
branch
from
533c1ed to
9ab227e
Compare
Signed-off-by: Michael Carroll <[email protected]>
azeey
approved these changes
Jul 19, 2023
Thanks @mjcarroll |
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 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)