Skip to content
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

Integration of depth estimation #19

Merged
merged 25 commits into from
Jul 26, 2019
Merged

Integration of depth estimation #19

merged 25 commits into from
Jul 26, 2019

Conversation

mihaibujanca
Copy link
Owner

  • Add case for flame library
  • Add depth estimation metric
  • Add Sophus as a dependency
  • Style improvements
  • Added visualisation for pixelformat::D_I_16 to view depth frames

@matthewspear
Copy link
Collaborator

Tried to build and run code at 84e56da

To reproduce:

make deps
make flame
make slambench APPS=flame

On the last command error occured:

-- Generation of flame-library : Activated
OpenCV LIBS: opencv_coreopencv_highguiopencv_imgprocopencv_imgcodecs
OpenCV Includes: /usr/include/usr/include/opencv
PCL INCUDES: 
-- Load version original of flame.
CMake Error at benchmarks/CMakeLists.txt:80 (ADD_SUBDIRECTORY):
  The binary directory

    /home/mspear/slambench2/build/benchmarks/flame/src/original

  is already used to build a source directory.  It cannot be used to build
  source directory

    /home/mspear/slambench2/benchmarks/flame/src/original

  Specify a unique binary directory name.
Call Stack (most recent call first):
  benchmarks/flame/CMakeLists.txt:2 (explore_implementations)


-- Configuring incomplete, errors occurred!
See also "/home/mspear/slambench2/build/CMakeFiles/CMakeOutput.log".
See also "/home/mspear/slambench2/build/CMakeFiles/CMakeError.log".
Makefile:180: recipe for target 'build/Makefile' failed
make: *** [build/Makefile] Error 1

Provided standard out and error:
stderr.txt
stdout.txt

@@ -10,7 +10,7 @@ ${REPOS_DIR}/ceres :

${DEPS_DIR}/ceres : ${REPOS_DIR}/ceres eigen3 suitesparse
mkdir ${REPOS_DIR}/ceres/build -p
rm ${REPOS_DIR}/ceres/buid/* -rf
rm ${REPOS_DIR}/ceres/build/* -rf
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ha, I spotted that too :)

@mihaibujanca mihaibujanca merged commit 9391d91 into master Jul 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants