Remove X11 call from Gazebo 7

Approved-by: Steve Peters <[email protected]>
Approved-by: Ian Chen <[email protected]>

Signed-off-by: Louise Poubel <[email protected]>

Signed-off-by: Ian Chen <[email protected]>

Signed-off-by: Ian Chen <[email protected]>

Signed-off-by: Ian Chen <[email protected]>

Signed-off-by: Ian Chen <[email protected]>

Signed-off-by: Ian Chen <[email protected]>

Signed-off-by: Ian Chen <[email protected]>

Signed-off-by: Ian Chen <[email protected]>

…#2769)

* Fixed crash when collision size is zero

Signed-off-by: ahcorde <[email protected]>

* Using ignition::math::isnan

Signed-off-by: ahcorde <[email protected]>

* Improved error message

Signed-off-by: ahcorde <[email protected]>

* fixed method to get the name of the visual

Signed-off-by: ahcorde <[email protected]>

* Fixed else brackets

Signed-off-by: ahcorde <[email protected]>

* [Gazebo 9] Added test to check collisions equal to zero (gazebosim#2788)

* Added test to check collisions equal to zero

Signed-off-by: ahcorde <[email protected]>

* Included feedback

Signed-off-by: ahcorde <[email protected]>

* make linters happy

Signed-off-by: ahcorde <[email protected]>

* Update Visual_TEST.cc

Signed-off-by: Louise Poubel <[email protected]>

Signed-off-by: Louise Poubel <[email protected]>

Signed-off-by: Ian Chen <[email protected]>

Backport lockstep changes to gazebo9

Signed-off-by: Louise Poubel <[email protected]>

Signed-off-by: Louise Poubel <[email protected]>

* add publisher private class to hold OnPublishComplete callback

Signed-off-by: Ian Chen <[email protected]>

* std bind and rearrage pub node fini

Signed-off-by: Ian Chen <[email protected]>

* add comment on PublisherPrivate

Signed-off-by: Ian Chen <[email protected]>

Signed-off-by: Louise Poubel <[email protected]>

…ers (gazebosim#2809)

Signed-off-by: Alejandro Hernández <[email protected]>

Co-authored-by: Jose Luis Rivero <[email protected]>
Co-authored-by: Louise Poubel <[email protected]>
Co-authored-by: Steve Peters <[email protected]>

Signed-off-by: ahcorde <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Co-authored-by: Alejandro Hernández Cordero <[email protected]>

…<p> (gazebosim#2825)

* add test to check gazebo model generated by assimp4, see gazebosim#2682

* Length of polylist <p> is not the number of inputs
(i.e. VERTEX and NORMAL) * vcount.size() * 3.
If both VERTEX and NORMAL have offset 0, then the length of
polylist <p> is vcount.size() * 3.
So use length of set(offset) as inputSize.
This fix for cases where VERTEX and NORMAL share same vcounts, see gazebosim#2682 (comment)

Signed-off-by: Steve Peters <[email protected]>
Co-authored-by: Steve Peters <[email protected]>

…m#2779)

When an absolute path is desired, the CMAKE_INSTALL_FULL_*DIR variables should
be used instead of concatenating with CMAKE_INSTALL_PREFIX. Special handling is
also needed for pkg-config files.

gazebosim#2833)

Signed-off-by: Ian Chen <[email protected]>

Signed-off-by: Ian Chen <[email protected]>

Signed-off-by: ahcorde <[email protected]>
Signed-off-by: Ian Chen <[email protected]>

Co-authored-by: Ian Chen <[email protected]>

Signed-off-by: Louise Poubel <[email protected]>

Signed-off-by: Louise Poubel <[email protected]>

…m#2836)

Signed-off-by: ahcorde <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>

Co-authored-by: Louise Poubel <[email protected]>

…im#2856)

Signed-off-by: Ian Chen <[email protected]>

Disable some code in SimpleTrackedVehiclePlugin for boost 1.74
or greater due to changes in boost shared_ptr.hpp

Signed-off-by: Steve Peters <[email protected]>

* fix physics based sensor update rate in lockstep mode

Signed-off-by: Ian Chen <[email protected]>

* merge lines and update changelog

Signed-off-by: Ian Chen <[email protected]>

Signed-off-by: Ian Chen <[email protected]>

Mainly adds tests to make sure we continue to support files with spaces.

Support encoded file names inside COLLADA files.

Signed-off-by: Louise Poubel <[email protected]>

Signed-off-by: Louise Poubel <[email protected]>

Signed-off-by: Louise Poubel <[email protected]>

Updated gazebo to generate shadow maps for spot lights.
This does not mean the spot lights will now start casting
shadows in gazebo - we still need to update the
RTShaderSystem code to actually render them.
This does however mean that the shadow maps are now
available and we can access them through custom
material shaders.

The spotlight_shadow_demo.world contains a shadow
receiver ground plane with custom material shaders to show
how to access the shadow texture generated by OGRE.

Signed-off-by: Ian Chen <[email protected]>

* Add test for performance metrics deadlock

Load the pr2 world with --lockstep and then subscribe to
/gazebo/performance_metrics to reproduce the deadlock.

* Fix deadlock when publishing performance metrics

Currently the PublishPerformanceMetrics function is called within
SensorManager.cc by each SensorContainer with protection by the
mutex of each SensorContainer. This leads to a deadlock when a
world has sensors in more than one container (i.e. more than one
category of sensors) as described in gazebosim#2902.

Since the PublishPerformanceMetrics function is not a member
function of SensorContainer and uses only global variables,
the function can be called by the SensorManager thread instead
of by each SensorContainer thread. This change will avoid the
deadlock.

Signed-off-by: Steve Peters <[email protected]>

gazebosim#2918)

* add support for camera with floating point image format
* visualize 16 bit float images

Signed-off-by: Ian Chen <[email protected]>
Signed-off-by: Steve Peters <[email protected]>
Co-authored-by: Steve Peters <[email protected]>

Signed-off-by: Steve Peters <[email protected]>

Signed-off-by: Ian Chen <[email protected]>

* Added color setting to LensFlare and LensFlareSensorPlugin.
Added methods to expose scale and color to derived plugins.
* Removed unused animated lens flare effect.
* Removed noise_rgba.png, which is no longer used.
* Changed a value in the lens flare shader back to its original 1.0.
This means there will be no compensation for the removal of the
legacy animation effect and the lens flares will look less intense.

Signed-off-by: Martin Pecka <[email protected]>

* Add LensFlarePrivate setter methods and move
  LensFlareCompositorListener defintion to header

* Modified LensFlareCompositorListener members to follow PIMPL

* Reduce line length and fix whitespace

Signed-off-by: Steve Peters <[email protected]>

Co-authored-by: kbjeppes <[email protected]>
Co-authored-by: Steve Peters <[email protected]>

…ameras (gazebosim#2972)

* Remove modifications to the shadow technique done in the
DepthCamera, this is not necessary and breaks lighting for all
other cameras that will render after it in the same frame

* Add call to setShadowsEnabled for the normals viewport as well

Co-authored-by: Steve Peters <[email protected]>

This is an adjustment to API added in gazebosim#2965, which is allowed
because the API has not yet been released.

Signed-off-by: Steve Peters <[email protected]>

Signed-off-by: Ivan Santiago Paunovic <[email protected]>

Signed-off-by: Ivan Santiago Paunovic <[email protected]>

Co-authored-by: Steve Peters <[email protected]>

* Add debugging output to ogre_log test
* ogre_log test: ignore GLX extension line

Signed-off-by: Steve Peters <[email protected]>

Signed-off-by: Steve Peters <[email protected]>

…im#2988)

* Add nullptr guard in TopicManager PubToSub
* Add regression test for 2875
* Port API to gazebo9

Signed-off-by: Emerson Knapp <[email protected]>
Signed-off-by: Jose Luis Rivero <[email protected]>

…im#2991)

* Add error message to RT shader
* Improve error message for loading worlds/empty.world

Signed-off-by: Audrow Nash <[email protected]>

… the same name (Gazebo9) (gazebosim#3016)

* Initial implementation of SDF checking feature

Co-authored-by: Jenn Nguyen <[email protected]>

* Update version
* Update Changelog

…gazebosim#3039) (gazebosim#3053)

* Fixed ApplyWrenchDialog to properly handle nested models.

* Fix improper link being selected when opening applywrenchdialog

* Fixed styling issues

* Test nested models in ApplyWrenchDialog + update test world

* Separate nested_multilink world to avoid breaking ModelListWidget test

* Fix incorrect comment

* More styling fixes

* Fix link name parsing to support sdformat<11

Co-authored-by: Cameron Miller <[email protected]>

…#3031)

* get ids recursively

* make namespacing of ~/model/modify, ~/model/info, and server-side name setting on modify consistent

* add tests for included and nested models

* get scoped name by stripping parent name

* move test models, add strip name test, style changes

Signed-off-by: Ian Chen <[email protected]>

Co-authored-by: Ian Chen <[email protected]>

* IncrementalPlot.cc: include qwt_scale_map.h
* Use Qt::MiddleButton to fix deprecation warning
* PlotCurve: add private helper for renamed members

In qwt 6.2, they have renamed all member variables,
so create helper functions to minimize the ifdefs we will need
to support both versions.

* PlotCurve: add ifdefs to support qwt 6.2.0

Signed-off-by: Steven Peters <[email protected]>

We currently don't support 2021 and newer, see gazebosim#2867.

Signed-off-by: Steven Peters <[email protected]>

Signed-off-by: Ian Chen <[email protected]>

- Initial connection (insert) and cleanup (remove) locked with mutex
- Signal (access) checks for object not null before attempting to access (not locking because would need recursive mutex)
- Declare mutex as mutable so EventCount const method can block it
- Include race condition test to play with connections inside an event

(contribution by Sonia Jin from AWS, back port of gazebosim#3042)
Co-authored-by: Jose Luis Rivero <[email protected]>
Co-authored-by: sonia <[email protected]>

Backport gazebosim#3031 - Make links within nested models modifiable from GUI Client

Signed-off-by: Jose Luis Rivero <[email protected]>

Co-authored-by: Steve Peters <[email protected]>

Signed-off-by: Martin Pecka <[email protected]>

Co-authored-by: Louise Poubel <[email protected]>

…gazebosim#3278)

* Use find_package(Python3) with newer cmake

Signed-off-by: Steve Peters <[email protected]>