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

5 ➡️ 6 #1199

Merged
merged 12 commits into from
Nov 16, 2021
Merged

5 ➡️ 6 #1199

merged 12 commits into from
Nov 16, 2021

Conversation

chapulina
Copy link
Contributor

➡️ Forward port

Port ign-gazebo5 to ign-gazebo6

Branch comparison: ign-gazebo6...gazebo5

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

peci1 and others added 8 commits November 10, 2021 14:58
Adds support for dynamic setting of joint limits (position, velocity and effort).

Signed-off-by: Martin Pecka <peckama2@fel.cvut.cz>
Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
Co-authored-by: Addisu Z. Taddese <addisu@openrobotics.org>
Adds support for tracked vehicles along the lines of https://osrf-migration.github.io/gazebo-gh-pages/#!/osrf/gazebo/pull-requests/2652/page/1 .

It is based on M. Pecka, K. Zimmermann and T. Svoboda, "Fast simulation of vehicles with non-deformable tracks," 2017 IEEE/RSJ International Conference on Intelligent Robots and Systems (IROS), 2017, pp. 6414-6419, doi: 10.1109/IROS.2017.8206546.


Signed-off-by: Martin Pecka <peckama2@fel.cvut.cz>
Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
Co-authored-by: Addisu Z. Taddese <addisu@openrobotics.org>
)

Spurious <plugin> elements were created when saving worlds because sdf::Element::GetElement was used to check if a <plugin> child element exists. However this function always creates a new element if the element does not exist instead of returning a nullptr. The solution is to use sdf::Element::FindElement instead.

Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
Signed-off-by: Louise Poubel <louise@openrobotics.org>
Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
Signed-off-by: Louise Poubel <louise@openrobotics.org>
Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
Signed-off-by: Louise Poubel <louise@openrobotics.org>
@github-actions github-actions bot added the 🏯 fortress Ignition Fortress label Nov 13, 2021
@codecov
Copy link

codecov bot commented Nov 13, 2021

Codecov Report

Merging #1199 (cff043e) into ign-gazebo6 (e4e36a7) will increase coverage by 0.40%.
The diff coverage is 72.01%.

Impacted file tree graph

@@               Coverage Diff               @@
##           ign-gazebo6    #1199      +/-   ##
===============================================
+ Coverage        61.47%   61.88%   +0.40%     
===============================================
  Files              268      274       +6     
  Lines            21626    22441     +815     
===============================================
+ Hits             13294    13887     +593     
- Misses            8332     8554     +222     
Impacted Files Coverage Δ
...stems/joint_state_publisher/JointStatePublisher.hh 100.00% <ø> (ø)
...stems/joint_state_publisher/JointStatePublisher.cc 81.13% <61.53%> (-3.25%) ⬇️
src/systems/tracked_vehicle/TrackedVehicle.cc 64.77% <64.77%> (ø)
src/systems/track_controller/TrackController.cc 70.94% <70.94%> (ø)
src/systems/physics/Physics.cc 71.70% <78.98%> (+0.87%) ⬆️
src/systems/thruster/Thruster.cc 92.07% <90.90%> (ø)
include/ignition/gazebo/components/Collision.hh 100.00% <100.00%> (ø)
...ignition/gazebo/components/JointEffortLimitsCmd.hh 100.00% <100.00%> (ø)
...nition/gazebo/components/JointPositionLimitsCmd.hh 100.00% <100.00%> (ø)
...nition/gazebo/components/JointVelocityLimitsCmd.hh 100.00% <100.00%> (ø)
... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ad6b9e2...cff043e. Read the comment docs.

chapulina and others added 4 commits November 15, 2021 09:36
Signed-off-by: Louise Poubel <louise@openrobotics.org>
Signed-off-by: Louise Poubel <louise@openrobotics.org>
Signed-off-by: Jasmeet Singh <jasmeet0915@gmail.com>
Signed-off-by: Louise Poubel <louise@openrobotics.org>
@chapulina chapulina merged commit 97f2e9c into ign-gazebo6 Nov 16, 2021
@chapulina chapulina deleted the chapulina/5_to_6 branch November 16, 2021 01:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏯 fortress Ignition Fortress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants