-
-
Notifications
You must be signed in to change notification settings - Fork 8
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
Gazebo ogre 1.12 problems #88
Comments
Relevant comment: gazebosim/gazebo-classic#2700 (comment) . |
Unfortunately, these issues still persist and block us from using Gazebo with ros-humble (see RoboStack/ros-humble#24). As far as I know, ros2 requires ogre 1.12. Even with the latest builds, when running |
On linux, the error is
/cc @traversaro @Nicogene |
In the next days I will try look into this again, also considering gazebosim/gazebo-classic#2700 (comment) that seems useful. |
Hi @traversaro, did you have a chance to look into this? |
Hi @Tobias-Fischer, I tried to set up the dev environment for this some time ago but compiling Gazebo locally is quite slow. :D The only modifications I was able to do is to try to do what was suggested in gazebosim/gazebo-classic#2700 (comment) at the CMake level, here a quick patch: diff --git a/media/CMakeLists.txt b/media/CMakeLists.txt
index 8c651154c0..033252a8c7 100644
--- a/media/CMakeLists.txt
+++ b/media/CMakeLists.txt
@@ -5,11 +5,29 @@ add_subdirectory(gui)
add_subdirectory(materials)
add_subdirectory(models)
-# use glsl 150 shaders if ogre version >= 1.9
-if ((NOT ${OGRE_VERSION} VERSION_LESS 1.9.0) AND NOT APPLE)
- add_subdirectory(rtshaderlib150)
-else ()
- add_subdirectory(rtshaderlib)
+option(GAZEBO_INSTALL_SHADERS_FROM_OGRE "If ON, do not install custom shader and only copy the one available in OGRE" OFF)
+if(NOT GAZEBO_INSTALL_SHADERS_FROM_OGRE)
+ # use glsl 150 shaders if ogre version >= 1.9
+ if ((NOT ${OGRE_VERSION} VERSION_LESS 1.9.0) AND NOT APPLE)
+ add_subdirectory(rtshaderlib150)
+ else ()
+ add_subdirectory(rtshaderlib)
+ endif()
+else()
+ if(NOT PKG_CONFIG_EXECUTABLE)
+ message(FATAL_ERROR "GAZEBO_INSTALL_SHADERS_FROM_OGRE option requires that pkg-config is available")
+ endif()
+ execute_process(COMMAND ${PKG_CONFIG_EXECUTABLE} --variable=prefix OGRE
+ OUTPUT_VARIABLE ogre_install_prefix
+ RESULT_VARIABLE ogre_install_prefix_result)
+ message(STATUS "=====> ogre_install_prefix: ${ogre_install_prefix}")
+ # See https://github.com/gazebosim/gazebo-classic/issues/2700#issuecomment-1041417601
+ file(GLOB rtshader_glsl_from_ogre "${ogre_install_prefix}/share/OGRE/Media/RTShaderLib/GLSL/*")
+ install(FILES ${rtshader_glsl_from_ogre}
+ DESTINATION ${CMAKE_INSTALL_PREFIX}/share/gazebo-${GAZEBO_MAJOR_VERSION}/media/rtshaderlib)
+ file(GLOB shadowvolume_files_from_ogre "${ogre_install_prefix}/share/OGRE/Media/ShadowVolume/*")
+ install(FILES ${shadowvolume_files_from_ogre}
+ DESTINATION ${CMAKE_INSTALL_PREFIX}/share/gazebo-${GAZEBO_MAJOR_VERSION}/media)
endif()
add_subdirectory(skyx) However, I still need to test this. Furthermore, even if everything works fine, this will result in no shading at all (basically shapes with flat color) so to actually make it usuable something else needs to be fixed. |
Ah, I see. This is an annoying issue for RoboStack - because of rviz2 we are tied to ogre 1.12. Someone just dropped by my office and told me they moved away from RoboStack because of they could not use Gazebo :(. I wonder what’s easier - debugging this issue here, or switching back to rviz_ogre_vendor and build the rest of RoboStack with ogre 1.10?!? Sorry to abuse this issue, happy to discuss elsewhere. Cc @wolfv |
:(
Despite I all in for de-vendoring, make stuff work with later version of the libraries, etc etc I am afraid switching back to rviz_ogre_vendor may be the easier option, at least in the short term. We have been struggling with this for a long time, and it is quite difficult to have an ETA on when this could be, for several reasons. |
We removed ogre 1.12 builds in #164 . |
The build added in #85 have some problems. They have been historically tracked in gazebosim/gazebo-classic#2700, I just opened this issue for giving publicity of that debugging activity.
The text was updated successfully, but these errors were encountered: