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.

Sign up for GitHub

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

Jump to bottom

3 ➡️ 4 #964

Merged
merged 16 commits into from
Aug 11, 2021
Merged

3 ➡️ 4 #964

merged 16 commits into from
Aug 11, 2021

Conversation

chapulina
Copy link
Contributor

➡️ Forward port

Port ign-gazebo3 to ign-gazebo4

Branch comparison: ign-gazebo4...ign-gazebo3

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

diegoferigo and others added 13 commits August 2, 2021 10:39
@diegoferigo
Fix logic to disable server default plugins loading (#953)
ad27b27 
Signed-off-by: Diego <diego.ferigo@iit.it>
@chapulina
Expose a test fixture helper class (#926)
8620011 
Signed-off-by: Louise Poubel <louise@openrobotics.org>
Signed-off-by: Arjo Chakravarty <arjo@openrobotics.org>
@herobank110 @chapulina
Add 25percent darker view angle icons (#426)
0431f9e 
Signed-off-by: David Kanekanian <dkanekanian1@gmail.com>
@joxoby @chapulina
No install apt recommends and clear cache (#423)
a299095 
Signed-off-by: Juan Oxoby <juan@vicarious.com>

Co-authored-by: Juan Oxoby <juan@vicarious.com>
Co-authored-by: Louise Poubel <louise@openrobotics.org>
@j-rivero @chapulina
Fix visibility macro names when used by a different component (Window…
bcef509 
…s) (#564)

Signed-off-by: Jose Luis Rivero <jrivero@osrfoundation.org>
@j-rivero @chapulina
Add missing IGNITION_GAZEBO_VISIBLE macros (#563)
dc50b29 
Signed-off-by: Jose Luis Rivero <jrivero@osrfoundation.org>
@chapulina
Examples and tutorial on using rendering API from plugins (#596)
7f711e0 
Signed-off-by: Louise Poubel <louise@openrobotics.org>

Edifice to Citadel (#945)

Signed-off-by: Louise Poubel <louise@openrobotics.org>
@chapulina
Setting the intiial velocity for a model or joint (#693)
c093937 
Signed-off-by: Louise Poubel <louise@openrobotics.org>
@ammaar8 @chapulina
Add UserCommands Plugin. (#719)
17aa8f5 
Signed-off-by: Ammaar Solkar <asketch8@gmail.com>
@chapulina @ahcorde
Fix component inspector shutdown crash (#724)
0f49361 
Signed-off-by: Louise Poubel <louise@openrobotics.org>

Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
@doisyg @chapulina
[DiffDrive] add enable/disable (#772)
a83f56f 
* add enable/disable diffdrive

Signed-off-by: Guillaume Doisy <guillaume.doisy@wyca.fr>

* remove debug

Signed-off-by: Guillaume Doisy <guillaume.doisy@wyca.fr>

* do not subscribe to enable if topic is empty

Signed-off-by: Guillaume Doisy <guillaume.doisy@wyca.fr>

* add test

Signed-off-by: Guillaume Doisy <guillaume.doisy@wyca.fr>

* lint and style

Signed-off-by: Guillaume Doisy <guillaume.doisy@wyca.fr>

* change enable type to bool and renamed to enabled

Signed-off-by: Guillaume Doisy <guillaume.doisy@wyca.fr>
@traversaro
Set protobuf_MODULE_COMPATIBLE before any find_package call (#957)
c57b3c9 
Signed-off-by: Silvio Traversaro <silvio.traversaro@iit.it>
@jennuine @mjcarroll
Drag and drop meshes into scene (#939)
b7346a4 
Signed-off-by: Jenn Nguyen <jenn@openrobotics.org>

Co-authored-by: Michael Carroll <michael@openrobotics.org>
chapulina added a commit that referenced this pull request Aug 10, 2021
@chapulina
Updates from 3 to 4 (#964))
a42ec3a 
Signed-off-by: Louise Poubel <louise@openrobotics.org>
@chapulina
Be more specific when looking for physics plugins (#965)
0f58670 
Signed-off-by: Louise Poubel <louise@openrobotics.org>
ahcorde
ahcorde approved these changes Aug 11, 2021
View reviewed changes
Copy link
Contributor

@ahcorde ahcorde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is there something wrong with Githb Actions ?

@chapulina
Copy link
Contributor Author

chapulina commented Aug 11, 2021 •
edited
Loading

is there something wrong with Githb Actions ?

Yeah GitHub was weird yesterday. Let me try to re-trigger

@chapulina
3 -> 4 (#964)
bec5f8a 
Signed-off-by: Louise Poubel <louise@openrobotics.org>
@chapulina
3 -> 4 fixes (#964)
e95f17c 
Signed-off-by: Louise Poubel <louise@openrobotics.org>
@codecov
Copy link

codecov bot commented Aug 11, 2021 •
edited
Loading

Codecov Report

Merging #964 (e95f17c) into ign-gazebo4 (2105ae3) will increase coverage by 0.04%.
The diff coverage is 68.49%.

Impacted file tree graph

@@               Coverage Diff               @@
##           ign-gazebo4     #964      +/-   ##
===============================================
+ Coverage        66.08%   66.13%   +0.04%     
===============================================
  Files              242      243       +1     
  Lines            18078    18205     +127     
===============================================
+ Hits             11946    12039      +93     
- Misses            6132     6166      +34
Impacted Files Coverage Δ
src/Conversions.cc 81.70% <ø> (ø)
src/ServerConfig.cc 89.68% <ø> (ø)
.../plugins/component_inspector/ComponentInspector.cc 7.09% <0.00%> (-0.15%) ⬇️
src/gui/plugins/scene3d/Scene3D.hh 66.66% <ø> (ø)
src/systems/air_pressure/AirPressure.hh 100.00% <ø> (ø)
src/systems/altimeter/Altimeter.hh 100.00% <ø> (ø)
src/systems/breadcrumbs/Breadcrumbs.hh 100.00% <ø> (ø)
src/systems/contact/Contact.hh 100.00% <ø> (ø)
src/systems/imu/Imu.hh 100.00% <ø> (ø)
src/systems/joint_controller/JointController.hh 100.00% <ø> (ø)
... and 23 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 2105ae3...e95f17c. Read the comment docs.

@chapulina chapulina merged commit 34eb437 into ign-gazebo4 Aug 11, 2021
@chapulina chapulina deleted the chapulina/3_to_4 branch August 11, 2021 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ahcorde ahcorde ahcorde approved these changes

@azeey azeey Awaiting requested review from azeey azeey is a code owner

@iche033 iche033 Awaiting requested review from iche033 iche033 is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
10 participants
@chapulina @ahcorde @diegoferigo @herobank110 @joxoby @j-rivero @ammaar8 @doisyg @traversaro @jennuine