Build issues: std::atomic and Qt::AA_DontCheckOpenGLContextThreadAffinity not defined about badslam HOT 8 CLOSED

Hi, the warnings in the CMake output can be ignored as long as you don't want to use RealSense or K4A live input. These libraries are optional.

The enum value Qt::AA_DontCheckOpenGLContextThreadAffinity has been added in Qt 5.8 according to the Qt documentation. It seems that you are likely using an older version? I will change the CMakeLists.txt file and ReadMe to require at least Qt 5.8.

To fix the error about std::atomic having an incomplete type, it should be sufficient to add #include <atomic> at the top of the file qt_thread.cc in which the error occurs. I will also fix that in the code.

from badslam.

@puzzlepaint Hi， Thank U for your reply! i have installed Qt 5.8.0 using (qt-opensource-linux-x64-5.8.0.run) from Official website. and also modified the CMakeLists.txt as "find_package(Qt5 5.8 REQUIRED COMPONENTS Core OpenGL Widgets)", but the same problem still occurs!! as follow: (May be my Qt 5.8.0 is not OK, can you give me some instruction for installing it?)

/home/gp/BAD-SLAM/badslam/libvis/src/libvis/opengl_context_qt.cc:126:46: error: ‘AA_DontCheckOpenGLContextThreadAffinity’ is not a member of ‘Qt’
QCoreApplication::instance()->setAttribute(Qt::AA_DontCheckOpenGLContextThreadAffinity);

Thank YOU very much!

from badslam.

@puzzlepaint it find Qt5.8 by "SET(Qt5_DIR "/home/gp/Qt5.8.0/5.8/gcc_64/lib/cmake/Qt5/")
SET(Qt5Test_DIR "/home/gp/Qt5.8.0/5.8/gcc_64/lib/cmake/Qt5Test")" in CMakeLists. but another problem occurs!!

from badslam.

@puzzlepaint, Dear friend, would you please write down the commands for the whole building process in sequence in Readme? Thank You very much!

from badslam.

It is also possible to set the Qt5_DIR and related variables via the CMake GUI or via ccmake. This avoids hardcoding them into CMakeLists.txt.

The ReadMe already contains all the build commands for the application. The errors that occur on your system arise from differing versions of the dependencies.

Regarding the new error (Eigen::half not found), it seems that fixing this requires to update the Eigen library. As stated in the ReadMe, the latest version (3.3.7) should definitely work, but I don't know what the minimum required version is.

from badslam.

@puzzlepaint My Dear friend, i have solved the problem, but there still be a bug in mine. as follow. would you please provide some advice to me? Thanks for your help so much! Additional, really want to know how do you install the Qt 5.8? also from the *.run file? (if not, could you please give me some Specific Instructions or commands step by step. so unfamiliar to Qt). thank you again!!!

from badslam.

This now looks like it might be an issue due to a too old version of the CUDA SDK. Which one do you use? Does it contain the __ushort_as_half and __half_as_ushort functions? All of the versions listed in the ReadMe should contain those.

I don't remember how I installed the up-to-date Qt, but I guess it probably was the .run file as well, yes.

from badslam.

Closing this for now due to no further activity.

from badslam.