LabelFusion icon indicating copy to clipboard operation
LabelFusion copied to clipboard

correct dockerfile & upgrade to nvidia-docker2

Open ianchen-tw opened this issue 7 years ago • 3 comments

Conclusion

  1. support nvidia-docker2
  2. a new dockerfile that can finally be built
  3. can run on ubuntu 18.04 now
  4. Add documentation for RealSense2 camera
  5. Add documentation for camera calibration
  6. fix some bugs

Bug fix

  1. the cdlf command can work now
  2. Support variable length of object-models

ianchen-tw avatar Nov 27 '18 13:11 ianchen-tw

Awesome, thanks for submitting this PR!

On Tue, Nov 27, 2018 at 8:34 AM Ian Chen [email protected] wrote:

Conclusion

  1. support nvidia-docker2
  2. a new dockerfile that can finally be built
  3. can run on ubuntu 18.04 now
  4. Add documentation for RealSense2 camera
  5. Add documentation for camera calibration
  6. fix some bugs

Bug fix

  1. the cdlf command can work now
  2. Support variable length of object-models

You can view, comment on, or merge this pull request online at:

https://github.com/RobotLocomotion/LabelFusion/pull/40 Commit Summary

  • fix alias cdlf
  • add env settings
  • change input number
  • expand nvidia-docker command into mult-line
  • seperate the install_dependcies.sh for readability
  • successfully compiled director
  • install elastic fusion's dependencies
  • compiled successfully
  • change docker run options
  • fix dependency for ubuntu 16.04
  • use the older version of director
  • add elasticfusion camera.cfg
  • add D435 camera parameter to render train image proccess
  • add intel realsense camera usage
  • Update README.rst
  • update realsense operation guide
  • Add camera calibration parameter doc
  • combine multiple RUN command and stop from sourcing other scripts
  • clean up code and use the latetest docker image
  • remove unnecessary files and add a config file
  • change the size of object to variable length
  • changes for pr

File Changes

Patch Links:

  • https://github.com/RobotLocomotion/LabelFusion/pull/40.patch
  • https://github.com/RobotLocomotion/LabelFusion/pull/40.diff

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/RobotLocomotion/LabelFusion/pull/40, or mute the thread https://github.com/notifications/unsubscribe-auth/AFYQqCllljpJOXpQF3W0zglzJQVNUZjEks5uzT9KgaJpZM4Y1f54 .

peteflorence avatar Nov 27 '18 13:11 peteflorence

hey! @patmarion It's all good now. just give it a look if you have time.

ianchen-tw avatar Dec 08 '18 07:12 ianchen-tw

Just adding a note for changes I had to make to get this to work with a GeForce GTX 1650 & Cuda compilation tools, release 8.0, V8.0.61 (from nvcc -V):

cd ~/ElasticFusion/Core/src/
vim CMakeLists.txt

delete the following line:
set(CUDA_ARCH_BIN "30 35 50 52 61" CACHE STRING "Specify 'real' GPU arch to build binaries for, BIN(PTX) format is supported. Example: 1.3 2.1(1.3) or 13 21(13)")

cd ../
rm -rf build && mkdir build && cd build && cmake ../src/ && make -j20

cd ../../GUI/
rm -rf build && mkdir build && cd build && cmake ../src/ && make -j20

In ElasticFusion build folder run (from sample data):
./ElasticFusion -l ~/labelfusion/data/logs/2017-06-13-29/original_log.lcmlog -f

or from ~/labelfusion/data/logs/2017-06-13-29/:
run_prep

akeaveny avatar Oct 22 '20 14:10 akeaveny