gazebo-classic:aditya/fix_dem_test_fail

Last commit made on 2023-02-09
Get this branch:
git clone -b aditya/fix_dem_test_fail https://git.launchpad.net/gazebo-classic

Branch merges

Branch information

Name:
aditya/fix_dem_test_fail
Repository:
lp:gazebo-classic

Recent commits

128a66c... by Steve Peters <email address hidden>

Merge branch 'gazebo11' into aditya/fix_dem_test_fail

e1f7f1f... by Terry Welsh <email address hidden>

Improvements to WideAngleCamera LensFlare bug fix (#3296)

* Reverse extraneous z-up to y-up transform that was corrupting
  WideAngleCamera LensFlare results. Fixed small related bugs.

* Use fisheye cameras in lensflare_wideangle_cam.world to
  improve testing of lens flare.

953badb... by Sanjuksha Nirgude <email address hidden>

Fix wide-angle camera lensflare occlusion (#3276)

* Fixing Z check in WideAngleCamera::Project3d

Signed-off-by: Sanjuksha Nirgude <email address hidden>

Added a test case for wideangle camera lensflare

Signed-off-by: Aditya <email address hidden>

Add extra model in test world

The pre-existing model in the test world exhibits
the lens flare rendering bug without the fix in
this branch, so I added an extra model at a different
pose that does not exhibit the bug. It makes it
easier to confirm that the bug is fixed.

* Add extra cameras with occluded views
* Add another camera with box to its side that
  is incorrectly occluded

Signed-off-by: Steve Peters <email address hidden>

08f01aa... by =?utf-8?q?Ond=C5=99ej_Svoboda?= <email address hidden>

Fix template specialization in constructors (#3295)

Fixes #3177.

86105ad... by Audrow Nash <email address hidden>

Set initial sim time from the command-line (#3294)

Signed-off-by: Audrow Nash <email address hidden>
Signed-off-by: Steve Peters <email address hidden>
Co-authored-by: Steve Peters <email address hidden>

cc87a67... by Silvio Traversaro <email address hidden>

Fix conda-forge GitHub Action CI by using mambaforge directly and by supporting graphviz >= 3 on Windows (#3287)

The fix is composed by two unrelated changes

### [Fix conda-forge GitHub Action CI by using mambaforge directly](https://github.com/gazebosim/gazebo-classic/commit/eea77c2d831763f6123b30e2b1a910b98205189a)

Currently the CI is failing on Linux and Windows with a weird conflict error. This is due to the fact that we are install mamba (from the conda-forge channel) on the top of miniconda3 (that by default install all the packages from the defaults channel). To make an analogy in apt world, this is like installing Debian, and then trying to install packages from the Ubuntu repo: something can go wrong.

To solve this problem, we install [mambaforge](https://github.com/conda-forge/miniforge) (a installer like miniconda3 but already using conda-forge packages) directly, so we always and only use packages from the conda-forge channel.

Similar to:
* https://github.com/robotology/robotology-superbuild/pull/840
* https://github.com/robotology/robometry/pull/141
* https://github.com/robotology-playground/yarp-devices-ros2/pull/44

### [Add support to use graphviz >= 3 on Windows](https://github.com/gazebosim/gazebo-classic/commit/15435b1520b7fbe041cb79c5f57829ab028eb5dc)

Since graphviz 3, any downstream library that on Windows links against a graphviz shared library needs to define the `GVDLL` preprocessor definition (see https://gitlab.com/graphviz/graphviz/-/blob/3.0.0/CHANGELOG.md#changed).

To deal with this, this PR adds the `GVDLL` definition for `gazebo_gui` when on Windows (on graphviz < 3 defining this definition will be harmless). As that prepocessor definition should not be set when linking static graphviz, and given that we can't detect in `FindGraphviz` if the linked graphviz is static or not, an ad-hoc CMake variable `GAZEBO_FINDGRAPHVIZ_USE_STATIC_GRAPHVIZ` is introduced in the case this definition needs to be disabled.

Similar to https://github.com/robotology/ycm/pull/414 .

c70761f... by Audrow Nash <email address hidden>

Fix typo in README (#3291)

Signed-off-by: Audrow Nash <email address hidden>

728840d... by Steve Peters <email address hidden>

msgs.cc: add missing <array> include (#3290)

Fixes homebrew build.

Signed-off-by: Steve Peters <email address hidden>

fb9e0f6... by Silvio Traversaro <email address hidden>

Remove legacy workarounds in conda-forge GitHub Actions job (#3270)

In https://github.com/gazebosim/gazebo-classic/pull/3223#issuecomment-1140038513 we added some workarounds for the conda-forge CI, namely pin the version of some dependencies and removed dartsim. Now that the long term fixes to that problems (linked in https://github.com/gazebosim/gazebo-classic/pull/3223#issuecomment-1140038513) has been fixed, we can remove the workarounds.

5fef8ee... by Steve Peters <email address hidden>

Fix crash with DEM and Camera (#3279)

* Test DEM heightmap with camera. This exposes a crash.
* rendering: ensure SphericalCoordinates != nullptr

Signed-off-by: Steve Peters <email address hidden>