-
Notifications
You must be signed in to change notification settings - Fork 790
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
Build issue on Arch Linux using ROS Melodic #425
Comments
Cmake output
|
This is due to a modification in the VTK lib. PCL is not properly including the vtkTexture header and instead relies on another header to have imported it beforehand. Since this is a PCL problem, this is not the place to fix it, but you can mitigate it by adding the vtkTexture import in CloudViewer.h, like so : diff --git a/guilib/include/rtabmap/gui/CloudViewer.h b/guilib/include/rtabmap/gui/CloudViewer.h
index 526f4bc2..56701b59 100644
--- a/guilib/include/rtabmap/gui/CloudViewer.h
+++ b/guilib/include/rtabmap/gui/CloudViewer.h
@@ -28,6 +28,8 @@ SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
#ifndef RTABMAP_CLOUDVIEWER_H_
#define RTABMAP_CLOUDVIEWER_H_
+#include <vtkTexture.h>
+
#include "rtabmap/gui/RtabmapGuiExp.h" // DLL export/import defines |
Thx @discord-intech for the hint. I've made the change in the commit above. I cannot test it on my testing computer (using PCL binaries, not latest version from source), let me know if this works. |
I tested it on my Archlinux machine, it should work for @parkerlreed too. |
Thanks! Will test compiling now |
@discord-intech Quick question before I submit another issue: If you view point cloud and then exit the window, does it also crash for you? It's weird because the main window pops up on top of the point cloud view. You can use it as much as you want, but as soon as you exit
|
Oh I realized it may be slightly related to the CloudViewer that was fixed hmm |
I cannot reproduce the same crash with my system. For info, I use i3wm, so it might be related to your DE and VTK having a spat and forcing Xorg to kill rtabmapviz. I do not think this a problem related to rtabmap, but I could be wrong. |
Ok Ill take a stab at it later. Now that you mention it, last time I used this a lot natively was on i3wm heh. |
@discord-intech Doesn't fully crash on i3 but does lock up completely. https://www.youtube.com/watch?v=V7-jl5ogS8Q vtk 8.2.0-5 |
Did you try with vtk 6.3.0 (default version on 18.04)? |
Currently building vtk6. Will try an rtabmap compile once that finishes. |
I tried with both set to the /opt/vtk6/cmake and it compiled but crashes out on start
|
You may look at this post: #427 (comment) On Arch Linux, Qt5 seems built with OpenGL ES instead of OpenGL, causing issues with vtk binaries that seem built for OpenGL, not OpenGL ES. rtabmap binaries then crash if the UI is used. The rtabmap node would be okay, so the workaround is to use RVIZ instead of rtabmapviz for visualization. |
Thanks, will take a look. That vtk6 was compiled from source so I would hope it would match the system libs/OpenGL support. Arch has tried pretty hard to move away from everything QT4. |
Installed the full melodic-desktop package and went to compile rtabmap.
Arch Linux
gcc 9.1.0-2
The text was updated successfully, but these errors were encountered: