mihaibujanca / dynamicfusion

Implementation of Newcombe et al. CVPR 2015 DynamicFusion paper
BSD 3-Clause "New" or "Revised" License
394 stars 105 forks source link

install with cmake, windows #33

Open mxgbs opened 6 years ago

mxgbs commented 6 years ago

Got the error message as following. Do you have any guide for building with cmake+VS2015. Thanks.

CMake Error at C:/Program Files/CMake/share/cmake-3.10/Modules/FindPackageHandleStandardArgs.cmake:137 (message): Could NOT find OpenCV (missing: viz) (found version "3.3.0") Call Stack (most recent call first): C:/Program Files/CMake/share/cmake-3.10/Modules/FindPackageHandleStandardArgs.cmake:378 (_FPHSA_FAILURE_MESSAGE) C:/Users/dengya/Documents/opencv_files/opencv/build/x64/vc14/lib/OpenCVConfig.cmake:261 (find_package_handle_standard_args) C:/Users/dengya/Documents/opencv_files/opencv/build/OpenCVConfig.cmake:122 (include) CMakeLists.txt:47 (find_package)

HaoguangHuang commented 6 years ago

Need to recmake and rebuild your opencv project manually. Your opencv lack the viz module.

mxgbs commented 6 years ago

Thank you. Have you installed in windows using cmake and vs? I have issue to build google-glog and suitesparse with cmake. Have you successfully build that? Ceres is not listed in the windows version's dependencies. However, when I tried to build the dynamic fusion, it said ceres is not found. Therefore, I think it does need cere pre-installed.

On Mon, Nov 20, 2017 at 11:08 PM, HaoguangHuang notifications@github.com wrote:

Need to recmake and rebuild your opencv project manually. Your opencv lack the viz module.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-345936423, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KC0spQTViKS8yjAu2ezjwyvpaXHOjks5s4nbjgaJpZM4QlCbp .

HaoguangHuang commented 6 years ago

I have just built the project dynamicfusion successfully on my ubuntu 16.04, but haven't tried on windows.

The latest version of this project seems to use Opt instead of Ceres, to process optimization of warp field.

mihaibujanca commented 6 years ago

I will need to update the documentation and building process. Probably going to find some time for it this weekend.

Indeed the current version is using Opt rather than Ceres, so you should be able to work around installing Ceres, but I don't think that the code is modular enough at the moment to let you just comment out the dependency in the CMakeLists

mihaibujanca commented 6 years ago

@HaoguangHuang Any chance you could write what was needed other than the current instructions to get it working on Windows?
I'll need to write this up for other users, would appreciate any suggestions on that

mxgbs commented 6 years ago

Hi,

Built the ceres and another error occurred when I tried to build dynamic fusion. Can I disable the use of OPENNI? I thought that's optional. Thank you.

CMake Error: The following variables are used in this project, but they are set to NOTFOUND. Please set them or make sure they are set and tested correctly in the CMake files: OPENNI_INCLUDE_DIR (ADVANCED) used as include directory in directory C:/Users/dengya/Documents/dynamicfusion used as include directory in directory C:/Users/dengya/Documents/dynamicfusion/kfusion used as include directory in directory C:/Users/dengya/Documents/dynamicfusion/kfusion used as include directory in directory C:/Users/dengya/Documents/dynamicfusion/apps used as include directory in directory C:/Users/dengya/Documents/dynamicfusion/apps OPENNI_LIBRARY (ADVANCED) linked by target "kfusion" in directory C:/Users/dengya/Documents/dynamicfusion/kfusion

On Wed, Nov 22, 2017 at 8:38 AM, Mihai notifications@github.com wrote:

@HaoguangHuang https://github.com/haoguanghuang Any chance you could write what was needed other than the current instructions to get it working on Windows? I'll need to write this up for other users, would appreciate any suggestions on that

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-346405663, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KCz_UeKWFeC43hV4VzUh7ZD7xRZ1Tks5s5E3wgaJpZM4QlCbp .

HaoguangHuang commented 6 years ago

@mihaibujanca If you need, I am pleased to supple some simple instructions here about how to build this project on ubuntu 16.04 soon.

HaoguangHuang commented 6 years ago

@mxgbs The solution contains two projects that can produce executable file, one is dynamicfusion, the other is dynamicfusion_kinect. And they correspond to demo.cpp and dynamicfusion_kinect.cpp respectively. If you have no OPENNI installed, don't choose ALL_BUILD when build the project. Just building 'dynamicfusion' is enough

mxgbs commented 6 years ago

Hi Haoguang,

Thanks for your kindly reply. Do you have any idea how can I disable the dynamicfusion_kinect part and openNI dependency in CMakeLists.txt? Thank you.

Sincerely,

Yan

On Wed, Nov 22, 2017 at 8:12 PM, HaoguangHuang notifications@github.com wrote:

@mxgbs https://github.com/mxgbs The solution contains two projects that can produce executable file, one is dynamicfusion, the other is dynamicfusion_kinect. And they correspond to demo.cpp and dynamicfusion_kinect.cpp respectively. If you have no OPENNI installed, don't choose ALL_BUILD when build the project. Just building 'dynamicfusion' is enough

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-346528747, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KCxlkUiL3tUzq07oc8PYOZgEnU90Uks5s5PDHgaJpZM4QlCbp .

mihaibujanca commented 6 years ago

One way is to just remove lines 18-27 from apps/CMakeLists.txt. If OpenNI is not installed, that shouldn't be built anyway - not sure why it appears to be a problem.

Alternatively just explicitly use set(OPENNI_FOUND FALSE) somewhere before add_subdirectory(apps) in the main CMakeLists.txt

mxgbs commented 6 years ago

Hi Mihai,

Thanks for your quick response. I tried, and use cmake, the same error still occurs:. I think the reason may be because the following functions using some OPENNI functions? CMake Error: The following variables are used in this project, but they are set to NOTFOUND. Please set them or make sure they are set and tested correctly in the CMake files: OPENNI_INCLUDE_DIR (ADVANCED) used as include directory in directory C:/Users/dengya/Documents/dynamicfusion used as include directory in directory C:/Users/dengya/Documents/dynamicfusion/kfusion used as include directory in directory C:/Users/dengya/Documents/dynamicfusion/kfusion used as include directory in directory C:/Users/dengya/Documents/dynamicfusion/apps used as include directory in directory C:/Users/dengya/Documents/dynamicfusion/apps OPENNI_LIBRARY (ADVANCED) linked by target "kfusion" in directory C:/Users/dengya/Documents/dynamicfusion/kfusion

Sincerely,

Yan

On Mon, Nov 27, 2017 at 1:05 PM, Mihai notifications@github.com wrote:

One way is to just remove lines 18-27 from apps/CMakeLists.txt. If OpenNI is not installed, that shouldn't be built anyway - not sure why it appears to be a problem.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-347327663, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KC3y53O1y7XiY6vMm-NGAJRhklfgXks5s6yQUgaJpZM4QlCbp .

mxgbs commented 6 years ago

I removed dynamicfusion_kinect from apps. The .sln is successfully generated via cmake. However, new error as following occurs when building with VS2015. Trying to solve that now.

1>------ Build started: Project: ZERO_CHECK, Configuration: Release x64

1> Checking Build System 1> CMake does not need to re-run because C:/Users/dengya/Documents/dynamicfusion/build_new/CMakeFiles/generate.stamp is up-to-date. 1> CMake does not need to re-run because C:/Users/dengya/Documents/dynamicfusion/build_new/kfusion/CMakeFiles/generate.stamp is up-to-date. 1> CMake does not need to re-run because C:/Users/dengya/Documents/dynamicfusion/build_new/apps/CMakeFiles/generate.stamp is up-to-date. 2>------ Build started: Project: kfusion, Configuration: Release x64 ------ 2> Building NVCC (Device) object kfusion/CMakeFiles/kfusion.dir/src/cuda/Release/kfusion_generated_imgproc.cu.obj 2> imgproc.cu 2>cl : Command line error D8021: invalid numeric argument '/Wno-write-strings'

On Mon, Nov 27, 2017 at 1:17 PM, yan deng dengyaner@gmail.com wrote:

Hi Mihai,

Thanks for your quick response. I tried, and use cmake, the same error still occurs:. I think the reason may be because the following functions using some OPENNI functions? CMake Error: The following variables are used in this project, but they are set to NOTFOUND. Please set them or make sure they are set and tested correctly in the CMake files: OPENNI_INCLUDE_DIR (ADVANCED) used as include directory in directory C:/Users/dengya/Documents/ dynamicfusion used as include directory in directory C:/Users/dengya/Documents/ dynamicfusion/kfusion used as include directory in directory C:/Users/dengya/Documents/ dynamicfusion/kfusion used as include directory in directory C:/Users/dengya/Documents/ dynamicfusion/apps used as include directory in directory C:/Users/dengya/Documents/ dynamicfusion/apps OPENNI_LIBRARY (ADVANCED) linked by target "kfusion" in directory C:/Users/dengya/Documents/ dynamicfusion/kfusion

Sincerely,

Yan

On Mon, Nov 27, 2017 at 1:05 PM, Mihai notifications@github.com wrote:

One way is to just remove lines 18-27 from apps/CMakeLists.txt. If OpenNI is not installed, that shouldn't be built anyway - not sure why it appears to be a problem.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-347327663, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KC3y53O1y7XiY6vMm-NGAJRhklfgXks5s6yQUgaJpZM4QlCbp .

mihaibujanca commented 6 years ago

Try removing -Wno-write-strings on line 5 of CMakeLists.txt

mxgbs commented 6 years ago

More errors come. For the mlibCore.h, should we install opt first and put that to the cmakelist? seems all the following libraries are missing.

include

include

include

include

include

1>------ Build started: Project: ZERO_CHECK, Configuration: Release x64

2>------ Build started: Project: kfusion, Configuration: Release x64 ------ 2> Building NVCC (Device) object kfusion/CMakeFiles/kfusion.dir/src/cuda/Release/kfusion_generated_imgproc.cu.obj 2> imgproc.cu 2>CUSTOMBUILD : nvcc warning : The -std=c++11 flag is not supported with the configured host compiler. Flag will be ignored. 2>c:\users\dengya\documents\dynamicfusion\kfusion\src\cuda\temp_utils.hpp(585): warning : function "ballot" 2> c:\program files\nvidia gpu computing toolkit\cuda\v9.0\include\sm_20_intrinsics.h(407): here was declared deprecated ("ballot() is deprecated in favor of ballot_sync() and may be removed in a future release (Use -Wno-deprecated-declarations to suppress this warning).") 2> 2>c:\users\dengya\documents\dynamicfusion\kfusion\src\cuda\temp_utils.hpp(597): warning : function "all" 2> c:\program files\nvidia gpu computing toolkit\cuda\v9.0\include\device_atomic_functions.h(181): here was declared deprecated ("all() is deprecated in favor of all_sync() and may be removed in a future release (Use -Wno-deprecated-declarations to suppress this warning).") 2> 2>c:\users\dengya\documents\dynamicfusion\kfusion\src\cuda\device.hpp(54): error : identifier "__float2half_rn" is undefined 2> 2>c:\users\dengya\documents\dynamicfusion\kfusion\src\cuda\device.hpp(59): error : identifier "half2float" is undefined 2> 2>c:\users\dengya\documents\dynamicfusion\kfusion\src\cuda\device.hpp(61): error : identifier "half2float" is undefined 2> 2>C:/Users/dengya/Documents/dynamicfusion/kfusion/src/cuda/imgproc.cu(270): error : identifier "__float2half_rn" is undefined 2> 2> 4 errors detected in the compilation of "C:/Users/dengya/AppData/Local/Temp/tmpxft_000061a0_00000000-13_imgproc.compute_61.cpp1.ii". 2>CUSTOMBUILD : nvcc warning : The -std=c++11 flag is not supported with the configured host compiler. Flag will be ignored. 2> imgproc.cu 2> CMake Error at kfusion_generated_imgproc.cu.obj.Release.cmake:279 (message): 2> Error generating file 2> C:/Users/dengya/Documents/dynamicfusion/build_new/kfusion/CMakeFiles/kfusion.dir/src/cuda/Release/kfusion_generated_imgproc.cu.obj 2> 2> 3>------ Build started: Project: dynamicfusion, Configuration: Release x64

3> demo.cpp 3>C:\Users\dengya\Documents\dynamicfusion\kfusion\src\utils\dual_quaternion.hpp(17): warning C4305: 'return': truncation from 'double' to 'float' 3>c:\users\dengya\documents\dynamicfusion\kfusion\include\opt\mLibInclude.h(15): fatal error C1083: Cannot open include file: 'mLibCore.h': No such file or directory 4>------ Build started: Project: ALL_BUILD, Configuration: Release x64

5>------ Skipped Build: Project: INSTALL, Configuration: Release x64 ------ 5>Project not selected to build for this solution configuration ========== Build: 2 succeeded, 2 failed, 0 up-to-date, 1 skipped ==========

On Mon, Nov 27, 2017 at 1:32 PM, Mihai notifications@github.com wrote:

Try removing -Wno-write-strings on line 5 of CMakeLists.txt

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-347335141, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KC6hGUP7vWsfM1hvKdCO_HkfkiLGvks5s6yp2gaJpZM4QlCbp .

mihaibujanca commented 6 years ago

yeah you have to install Opt (and its dependencies) and modify OPT_INCLUDE_DIRS and OPT_LIBRARY in CMakeLists

mxgbs commented 6 years ago

Do you have any clues how to fix following errors?

dynamicfusion\kfusion\src\cuda\device.hpp(54): error : identifier "__float2half_rn" is undefined dynamicfusion\kfusion\src\cuda\device.hpp(59): error : identifier "half2float" is undefined dynamicfusion\kfusion\src\cuda\device.hpp(61): error : identifier "half2float" is undefined dynamicfusion/kfusion/src/cuda/imgproc.cu(270): error : identifier "__float2half_rn" is undefined

Sincerely,

Yan

On Mon, Nov 27, 2017 at 1:48 PM, Mihai notifications@github.com wrote:

yeah you have to install Opt (and its dependencies) and modify OPT_INCLUDE_DIRS and OPT_LIBRARY in CMakeLists

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-347339364, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KCwGjNaYlpbX4hA0K116cJ9_hubwIks5s6y4fgaJpZM4QlCbp .

mxgbs commented 6 years ago

Besides, is this the opt you used https://software.sandia.gov/opt++/ ?

Sincerely,

Yan

On Mon, Nov 27, 2017 at 1:50 PM, yan deng dengyaner@gmail.com wrote:

Do you have any clues how to fix following errors?

dynamicfusion\kfusion\src\cuda\device.hpp(54): error : identifier "__float2half_rn" is undefined dynamicfusion\kfusion\src\cuda\device.hpp(59): error : identifier "half2float" is undefined dynamicfusion\kfusion\src\cuda\device.hpp(61): error : identifier "half2float" is undefined dynamicfusion/kfusion/src/cuda/imgproc.cu(270): error : identifier "__float2half_rn" is undefined

Sincerely,

Yan

On Mon, Nov 27, 2017 at 1:48 PM, Mihai notifications@github.com wrote:

yeah you have to install Opt (and its dependencies) and modify OPT_INCLUDE_DIRS and OPT_LIBRARY in CMakeLists

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-347339364, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KCwGjNaYlpbX4hA0K116cJ9_hubwIks5s6y4fgaJpZM4QlCbp .

mihaibujanca commented 6 years ago

https://github.com/niessner/Opt

mihaibujanca commented 6 years ago

The errors on undefined identifiers would come either from missing CUDA dependencies or incompatible CUDA version: http://docs.nvidia.com/cuda/cuda-math-api/group__CUDA__MATH__INTRINSIC__CAST.html

mxgbs commented 6 years ago

I use the latest CUDA version 9.0. Should I downgrade? First time using CUDA, lack of experience. Have you build this project on Windows successfully? If no one did, should I just give up and try other platform? I recalled you mentioned that you're preparing another tutorial for windows install, how is going? Thanks.

Sincerely,

Yan

On Mon, Nov 27, 2017 at 2:14 PM, Mihai notifications@github.com wrote:

The errors on undefined identifiers would come either from missing CUDA dependencies or incompatible CUDA version

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-347346589, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KC9qiGNJAC37CjM0dZUmX2Ozc1S4qks5s6zREgaJpZM4QlCbp .

mxgbs commented 6 years ago

Hi Mihai,

I think the problem is here" Unsupported Features General CUDA ‣ CUDA library. The built-in functions __float2half_rn() and __half2float() have been removed. Use equivalent functionality in the updated fp16 header file from the CUDA toolkit." in https://developer.download.nvidia.com/compute/cuda/9.0/Prod/docs/sidebar/CUDA_Toolkit_Release_Notes.pdf

Sincerely,

Yan

On Tue, Nov 28, 2017 at 2:22 PM, yan deng dengyaner@gmail.com wrote:

I use the latest CUDA version 9.0. Should I downgrade? First time using CUDA, lack of experience. Have you build this project on Windows successfully? If no one did, should I just give up and try other platform? I recalled you mentioned that you're preparing another tutorial for windows install, how is going? Thanks.

Sincerely,

Yan

On Mon, Nov 27, 2017 at 2:14 PM, Mihai notifications@github.com wrote:

The errors on undefined identifiers would come either from missing CUDA dependencies or incompatible CUDA version

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-347346589, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KC9qiGNJAC37CjM0dZUmX2Ozc1S4qks5s6zREgaJpZM4QlCbp .

mxgbs commented 6 years ago

Sorry. The last message is not correct. I saw all my device intrinsic functions defined by #if defined(CUDACC) are uncomment. That means CUDACCis not defined. But have no idea how to solve that. Google a lot and not find a solution.

Sincerely,

Yan

On Tue, Nov 28, 2017 at 2:32 PM, yan deng dengyaner@gmail.com wrote:

Hi Mihai,

I think the problem is here" Unsupported Features General CUDA ‣ CUDA library. The built-in functions __float2half_rn() and _half2float() have been removed. Use equivalent functionality in the updated fp16 header file from the CUDA toolkit." in https://developer.download. nvidia.com/compute/cuda/9.0/Prod/docs/sidebar/CUDA Toolkit_Release_Notes.pdf

Sincerely,

Yan

On Tue, Nov 28, 2017 at 2:22 PM, yan deng dengyaner@gmail.com wrote:

I use the latest CUDA version 9.0. Should I downgrade? First time using CUDA, lack of experience. Have you build this project on Windows successfully? If no one did, should I just give up and try other platform? I recalled you mentioned that you're preparing another tutorial for windows install, how is going? Thanks.

Sincerely,

Yan

On Mon, Nov 27, 2017 at 2:14 PM, Mihai notifications@github.com wrote:

The errors on undefined identifiers would come either from missing CUDA dependencies or incompatible CUDA version

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-347346589, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KC9qiGNJAC37CjM0dZUmX2Ozc1S4qks5s6zREgaJpZM4QlCbp .

mihaibujanca commented 6 years ago

Ok, just got hold of a windows machine, will try building it soon. Are you using windows 10?

mxgbs commented 6 years ago

Yes. Thank you. I try to build everything with VS2015, 64bits.

Sincerely,

Yan

On Wed, Nov 29, 2017 at 4:56 AM, Mihai notifications@github.com wrote:

Ok, just got hold of a windows machine, will try building it soon. Are you using windows 10?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-347852320, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KC5bP1F9fB2DFRWz5xE7msGgqYIimks5s7VRmgaJpZM4QlCbp .

mxgbs commented 6 years ago

Hi Mihai,

I have found out reason why following error occurs. The .sln generated by cmake is not automatically linked to CUDA. Need to manually configure build dependencies/build customizations and choose cuda 9.0. And then configure all the properties related to CUDA C/C++ in property page. dynamicfusion\kfusion\src\cuda\device.hpp(54): error : identifier "__float2half_rn" is undefined dynamicfusion\kfusion\src\cuda\device.hpp(59): error : identifier "half2float" is undefined dynamicfusion\kfusion\src\cuda\device.hpp(61): error : identifier "half2float" is undefined dynamicfusion/kfusion/src/cuda/imgproc.cu(270): error : identifier "__float2half_rn" is undefined

Sincerely,

Yan

On Wed, Nov 29, 2017 at 10:13 AM, yan deng dengyaner@gmail.com wrote:

Yes. Thank you. I try to build everything with VS2015, 64bits.

Sincerely,

Yan

On Wed, Nov 29, 2017 at 4:56 AM, Mihai notifications@github.com wrote:

Ok, just got hold of a windows machine, will try building it soon. Are you using windows 10?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-347852320, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KC5bP1F9fB2DFRWz5xE7msGgqYIimks5s7VRmgaJpZM4QlCbp .

mihaibujanca commented 6 years ago

Any clue why would that happen?

How do you solve that in practice / is there a way I could add that to the repo so it happens automatically?

I suppose I could try building a .sln and include it in the repo, but I'm quite clumsy with developing on windows

mxgbs commented 6 years ago

http://docs.nvidia.com/cuda/cuda-installation-guide-microsoft-windows/index.html#build-customizations-for-existing-projects

I firstly did the customization following 3.4. Build Customizations for Existing Projects http://docs.nvidia.com/cuda/cuda-installation-guide-microsoft-windows/index.html#build-customizations-for-existing-projects so that you can have ncvv as compiler. Otherwise, all the intrinsic functions would not be defined. More specifically for VS2015, right click opened project, choose build dependencies/build customizations and select the CUDA.

Then in the property page, CUDA C/C++ would be available after add dependency, like following shows. I then configures all the entries in CUDA C/C++ and Linker as those provided by CUDA sample code, e.g., C:\ProgramData\NVIDIA Corporation\CUDA Samples\v9.0\5_Simulations\nbody. Details are listed as following. However, I'm not sure if they are all essential. I guess there should be some way to configure all those things in CMakeLists. I don't have experience with CUDA development and thus don't know how to do that. Will try to find answer after I build Opt.

The entries entered/modified in CUDA C/C++ are: common/additional include directories common/target machine platform device/code generation host/preprocessor definition host/runtime library command line/ additional options

The entries in Linker: general/additional library directiories input/additional dependencies.

[image: Inline image 1]

Sincerely,

Yan

On Wed, Nov 29, 2017 at 1:46 PM, Mihai notifications@github.com wrote:

Any clue why would that happen?

How do you solve that in practice / is there a way I could add that to the repo so it happens automatically?

I suppose I could try building a .sln and include it in the repo, but I'm quite clumsy with developing on windows

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-348007039, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KC2CixI_nX7DMms9fprMfSJvOnb4Kks5s7dCwgaJpZM4QlCbp .

mxgbs commented 6 years ago

The following figure shows why functions would be shown as undefined if compiler nvcc is not linked. [image: Inline image 1]

On Wed, Nov 29, 2017 at 2:17 PM, yan deng dengyaner@gmail.com wrote:

http://docs.nvidia.com/cuda/cuda-installation-guide- microsoft-windows/index.html#build-customizations-for-existing-projects I firstly did the customization following 3.4. Build Customizations for Existing Projects http://docs.nvidia.com/cuda/cuda-installation-guide-microsoft-windows/index.html#build-customizations-for-existing-projects so that you can have ncvv as compiler. Otherwise, all the intrinsic functions would not be defined. More specifically for VS2015, right click opened project, choose build dependencies/build customizations and select the CUDA.

Then in the property page, CUDA C/C++ would be available after add dependency, like following shows. I then configures all the entries in CUDA C/C++ and Linker as those provided by CUDA sample code, e.g., C:\ProgramData\NVIDIA Corporation\CUDA Samples\v9.0\5_Simulations\nbody. Details are listed as following. However, I'm not sure if they are all essential. I guess there should be some way to configure all those things in CMakeLists. I don't have experience with CUDA development and thus don't know how to do that. Will try to find answer after I build Opt.

The entries entered/modified in CUDA C/C++ are: common/additional include directories common/target machine platform device/code generation host/preprocessor definition host/runtime library command line/ additional options

The entries in Linker: general/additional library directiories input/additional dependencies.

[image: Inline image 1]

Sincerely,

Yan

On Wed, Nov 29, 2017 at 1:46 PM, Mihai notifications@github.com wrote:

Any clue why would that happen?

How do you solve that in practice / is there a way I could add that to the repo so it happens automatically?

I suppose I could try building a .sln and include it in the repo, but I'm quite clumsy with developing on windows

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-348007039, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KC2CixI_nX7DMms9fprMfSJvOnb4Kks5s7dCwgaJpZM4QlCbp .

mxgbs commented 6 years ago

I tried to use terra released version for windows. However, many problem occurs. The including "stdio,h" problem can be solved by add terralib.includepath = [[C:\Program Files (x86)\Windows Kits\10\Include\10.0.16299.0\ucrt;C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\include;C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\crt\src]] to the terra script. However, when saveobj is using, e.g., terralib.saveobj("hello",{main = hello}), terra is still not working. Unluckily, that function is required for Opt to be built. Changing terralib.saveobj("hello",{main = hello}) to terralib.saveobj("hello.o",{main = hello}) would generate hello.o. However, when using g++ to build executable file, error popped as following. error LNK2019: unresolved external symbol acrt_iob_func referenced in function main error LNK2019: unresolved external symbol stdio_common_vfprintf referenced in function _vfprintf_l I finally found out the reason is VS2015 https://community.microfocus.com/microfocus/mainframe_solutions/enterprise_developer_21/w/knowledge_base/27927/compatibility-issues-with-visual-studio-2015. However, I still don't know how to solve that. Changing to other VS might bring more problem since all the other libraries I built are based on VS2015, .

On Wed, Nov 29, 2017 at 2:25 PM, yan deng dengyaner@gmail.com wrote:

The following figure shows why functions would be shown as undefined if compiler nvcc is not linked. [image: Inline image 1]

On Wed, Nov 29, 2017 at 2:17 PM, yan deng dengyaner@gmail.com wrote:

http://docs.nvidia.com/cuda/cuda-installation-guide-microsof t-windows/index.html#build-customizations-for-existing-projects I firstly did the customization following 3.4. Build Customizations for Existing Projects http://docs.nvidia.com/cuda/cuda-installation-guide-microsoft-windows/index.html#build-customizations-for-existing-projects so that you can have ncvv as compiler. Otherwise, all the intrinsic functions would not be defined. More specifically for VS2015, right click opened project, choose build dependencies/build customizations and select the CUDA.

Then in the property page, CUDA C/C++ would be available after add dependency, like following shows. I then configures all the entries in CUDA C/C++ and Linker as those provided by CUDA sample code, e.g., C:\ProgramData\NVIDIA Corporation\CUDA Samples\v9.0\5_Simulations\nbody. Details are listed as following. However, I'm not sure if they are all essential. I guess there should be some way to configure all those things in CMakeLists. I don't have experience with CUDA development and thus don't know how to do that. Will try to find answer after I build Opt.

The entries entered/modified in CUDA C/C++ are: common/additional include directories common/target machine platform device/code generation host/preprocessor definition host/runtime library command line/ additional options

The entries in Linker: general/additional library directiories input/additional dependencies.

[image: Inline image 1]

Sincerely,

Yan

On Wed, Nov 29, 2017 at 1:46 PM, Mihai notifications@github.com wrote:

Any clue why would that happen?

How do you solve that in practice / is there a way I could add that to the repo so it happens automatically?

I suppose I could try building a .sln and include it in the repo, but I'm quite clumsy with developing on windows

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-348007039, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KC2CixI_nX7DMms9fprMfSJvOnb4Kks5s7dCwgaJpZM4QlCbp .

electro-logic commented 6 years ago

I think that at this stage Ubuntu 16.04 should be the development environment for all to avoid losing time to support more platforms. Later when thirty party libraries will be better defined we should port it on Windows. I have written some instructions for compiling the project under Ubuntu, but now there are few changes like Opt instead of Ceres that I will update in instructions soon.

mxgbs commented 6 years ago

Solved former problem by using VS2015 x64 native tools instead of cmd for optbuild. Now compiled dynamicfusion with VS2015. Tons error comes. i. tgmath.h needs to be changed to . ii. if(event.action != cv::viz::KeyboardEvent::KEY_DOWN) problem with KEY_DOWN assignment. no clue how to change iii. bind is not defined which should be an internal function of std. Try adding using namespace std, not help. iv. tons of missing identifer or ";". Hope that's because the error introduced by ii, iii. Have no idea how to fix.

Sincerely,

Yan

[image: Inline image 1]

On Sat, Dec 2, 2017 at 8:44 AM, Leonardo notifications@github.com wrote:

I think that at this stage Ubuntu 16.04 should be the development environment for all to avoid losing time to support more platforms. Later when thirty party libraries will be better defined we should port it on Windows. I have written some instructions for compiling the project under Ubuntu, but now there are few changes like Opt instead of Ceres that I will update in instructions soon.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-348703894, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KC14JL769ILySPVkw8D7FftxAikEKks5s8X5XgaJpZM4QlCbp .

mihaibujanca commented 6 years ago

@mxgbs

  1. Got rid of it altogether, looks like it's not used anymore. Thanks :) Could you detail on 2, please?

  2. Bind might be CUDA actually, not std::bind. Where's the error coming from? I would suggest trying a lower version of cuda just to be sure, I'm currently using 8.

electro-logic commented 6 years ago

Opt conflict with Viz because KEY_UP, KEY_DOWN, etc.. are already defined in Viz. You can follow latest compiling instruction to see where to edit header files https://github.com/mihaibujanca/dynamicfusion/issues/19

mxgbs commented 6 years ago

@mihaibujanca @electro-logic I clone the latest version. But that error due to KEY_DOWN is still there. The place is demo.cpp 14, marked with I in the following complete error list. the bind error is related to this file: \opt\examples\external\mlib\include\core-network\networkServer.h Errors associated with that are marked with II.

There are also tons of errors came from opencv viz, marked with III.

The complete error list is as following: Severity Code Description Project File Line Suppression State I. Error C2059 syntax error: '::' dynamicfusion C:\Users\dengya\Documents\dynamic_fusion_git\dynamicfusion\apps\demo.cpp 14
II. Error C2440 '=': cannot convert from 'std::_Binder<std::_Unforced,SOCKET &,sockaddr &,int>' to 'int' kfusion c:\users\dengya\documents\optlang\opt\examples\external\mlib\include\core-network\networkServer.h 78
Error C2440 '=': cannot convert from 'std::_Binder<std::_Unforced,SOCKET &,sockaddr
&,int>' to 'int' kfusion c:\users\dengya\documents\optlang\opt\examples\external\mlib\include\core-network\networkServer.h 78

III. Error C2440 '=': cannot convert from 'std::_Binder<std::_Unforced,SOCKET &,sockaddr &,int>' to 'int' kfusion c:\users\dengya\documents\optlang\opt\examples\external\mlib\include\core-network\networkServer.h 78
Error C2440 '=': cannot convert from 'std::_Binder<std::_Unforced,SOCKET &,sockaddr
&,int>' to 'int' kfusion c:\users\dengya\documents\optlang\opt\examples\external\mlib\include\core-network\networkServer.h 78
Error C2440 '=': cannot convert from 'std::_Binder<std::_Unforced,SOCKET &,sockaddr &,int>' to 'int' kfusion c:\users\dengya\documents\optlang\opt\examples\external\mlib\include\core-network\networkServer.h 78
Error C2440 '=': cannot convert from 'std::_Binder<std::_Unforced,SOCKET &,sockaddr
&,int>' to 'int' kfusion c:\users\dengya\documents\optlang\opt\examples\external\mlib\include\core-network\networkServer.h 78
Error C2059 syntax error: 'constant' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\types.hpp 230 Error C2143 syntax error: missing ';' before '}' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\types.hpp 230 Error C2238 unexpected token(s) preceding ';' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\types.hpp 230 Error C2146 syntax error: missing ')' before identifier 'action' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\types.hpp 239 Error C2146 syntax error: missing ';' before identifier 'action' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\types.hpp 239 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\types.hpp 239 Error C2143 syntax error: missing ';' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\types.hpp 239 Error C2530 'cv::viz::symbol': references must be initialized kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\types.hpp 239 Error C2062 type 'unsigned char' unexpected kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\types.hpp 239 Error C2059 syntax error: ')' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\types.hpp 239 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\types.hpp 241 Error C2371 'cv::viz::Action': redefinition; different basic types kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\types.hpp 241 Error C2146 syntax error: missing ';' before identifier 'action' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\types.hpp 241 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\types.hpp 242 Error C2086 'int cv::viz::String': redefinition kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\types.hpp 242 Error C2146 syntax error: missing ';' before identifier 'symbol' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\types.hpp 242 Error C2059 syntax error: '}' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\types.hpp 275 Error C2143 syntax error: missing ';' before '}' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\types.hpp 275 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\types.hpp 282 Error C2448 'Scalar': function-style initializer appears to be a function definition kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\types.hpp 282 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\widgets.hpp 104 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\widgets.hpp 104 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\widgets.hpp 417 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\widgets.hpp 417 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\widgets.hpp 423 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\widgets.hpp 423 Error C3646 'getText': unknown override specifier kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\widgets.hpp 426 Error C2059 syntax error: '(' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\widgets.hpp 426 Error C2238 unexpected token(s) preceding ';' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\widgets.hpp 426 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\widgets.hpp 442 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\widgets.hpp 442 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\widgets.hpp 449 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\widgets.hpp 449 Error C3646 'getText': unknown override specifier kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\widgets.hpp 452 Error C2059 syntax error: '(' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\widgets.hpp 452 Error C2238 unexpected token(s) preceding ';' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\widgets.hpp 452 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 78
Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 78
Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 88
Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 88
Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 94
Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 94
Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 103 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 103 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 120 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 120 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 127 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 127 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 133 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 133 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 159 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 159 Error C3646 'getWindowName': unknown override specifier kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 190 Error C2059 syntax error: '(' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 190 Error C2238 unexpected token(s) preceding ';' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 190 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 200 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 200 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 297 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 297 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 323 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 323 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 340 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\viz3d.hpp 340 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\vizcore.hpp 103 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\vizcore.hpp 103 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\vizcore.hpp 109 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\vizcore.hpp 109 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\vizcore.hpp 149 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\vizcore.hpp 149 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\vizcore.hpp 150 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\vizcore.hpp 150 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\vizcore.hpp 155 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\vizcore.hpp 155 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\vizcore.hpp 160 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\vizcore.hpp 160 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\vizcore.hpp 161 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\vizcore.hpp 161 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\vizcore.hpp 164 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\vizcore.hpp 164 Error C4430 missing type specifier - int assumed. Note: C++ does not support default-int kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\vizcore.hpp 167 Error C2143 syntax error: missing ',' before '&' kfusion C:\Users\dengya\Documents\opencv-3.3.0\opencv-3.3.0\modules\viz\include\opencv2\viz\vizcore.hpp 167 Error C2440 '=': cannot convert from 'std::_Binder<std::_Unforced,SOCKET &,sockaddr *&,int>' to 'int' dynamicfusion c:\users\dengya\documents\optlang\opt\examples\external\mlib\include\core-network\networkServer.h 78

I. Error C2589 'constant': illegal token on right side of '::' dynamicfusion C:\Users\dengya\Documents\dynamic_fusion_git\dynamicfusion\apps\demo.cpp 14

mihaibujanca commented 6 years ago

@mxgbs

As per @electro-logic check instructions in #19. You need to edit opt/examples/external/mLib/include/mLibCore.h and remove #include "core-util/keycodes.h"

mihaibujanca commented 6 years ago

Hmm, try also commenting out #include "core-network/networkServer.h", same file. Looks like that might be conflicting as well, for II.

As for the opencv one, that's quite hard to debug from here, but it may have something to do with the way you compiled opencv being different from how you compile dynamicfusion

mxgbs commented 6 years ago

Thank you. All the problems are gone. Another eroor comes. So Openmesh is another dependency I need to install? LINK : fatal error LNK1181: cannot open input file '\home\mihai\Downloads\OpenMesh-6.3\build\Build\lib\libOpenMeshCore.a'

Sincerely,

Yan

On Tue, Dec 5, 2017 at 3:03 PM, Mihai notifications@github.com wrote:

Hmm, try also commenting out #include "core-network/networkServer.h", same file. Looks like that might be conflicting as well, for II

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-349470985, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KCwrWdhw49ctuQVbs6DgAPHoskzcFks5s9cvKgaJpZM4QlCbp .

mxgbs commented 6 years ago

@mihaibujanca @electro-logic Thank you. All the problems are gone. Another eroor comes. So Openmesh is another dependency I need to install? LINK : fatal error LNK1181: cannot open input file '\home\mihai\Downloads\OpenMesh-6.3\build\Build\lib\libOpenMeshCore.a'

mxgbs commented 6 years ago

Have got the project compiled. However, following two sentences reading nothing from input directories. I tried to change that two sentences to cv::glob("xxx/data/.*depth",depths), but still doesn't work. Try to fix that. Do you have any idea?

cv::glob(dir_name + "/depth", depths); cv::glob(dir_name + "/color", images);

On Tue, Dec 5, 2017 at 3:10 PM, yan deng dengyaner@gmail.com wrote:

Thank you. All the problems are gone. Another eroor comes. So Openmesh is another dependency I need to install? LINK : fatal error LNK1181: cannot open input file '\home\mihai\Downloads\ OpenMesh-6.3\build\Build\lib\libOpenMeshCore.a'

Sincerely,

Yan

On Tue, Dec 5, 2017 at 3:03 PM, Mihai notifications@github.com wrote:

Hmm, try also commenting out #include "core-network/networkServer.h", same file. Looks like that might be conflicting as well, for II

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-349470985, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KCwrWdhw49ctuQVbs6DgAPHoskzcFks5s9cvKgaJpZM4QlCbp .

mihaibujanca commented 6 years ago

I don't recall explicitly using openmesh in my code, I'll give it a check later and remove it if I can. OpenMesh is a dependency of Opt though. It may just have been me trying to use a more up to date version or something

mihaibujanca commented 6 years ago

@mxgbs Could you please provide file name and line number whenever you are referring to specific pieces of code? It'd be easier to get to them :)

Thanks

mihaibujanca commented 6 years ago

Have you definitely downloaded the data and checked that the paths are right?

mihaibujanca commented 6 years ago

Oh wait. That would be because you're not providing the path as an argument to the program, I imagine. you're supposed to run it using ./dynamicfusion path/to/data. Check the main README, Run instructions.

I must admit that they could be improved

mxgbs commented 6 years ago

I have input that and the variable Dir correctly shows that also.

On Tue, Dec 5, 2017 at 4:32 PM Mihai notifications@github.com wrote:

Oh wait. That would be because you're not providing the path as an argument to the program, I imagine. you're supposed to run it using ./dynamicfusion path/to/data.

Which reminds me I haven't documented that yet

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-349488989, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KC6P0SZ3rOc7rjPR7x4jyisxA82gLks5s9eCagaJpZM4QlCbp .

mxgbs commented 6 years ago

The line is 80-81 of demo.cpp. dir_name is my input path/to/data. Have you divided .depth and .color to two sub-folder, such as path-to-data/depth/.depth and path-to-data/color/.color? Though I don't think that's the problem.

Sincerely,

Yan cv::glob(dir_name + "/depth", depths); cv::glob(dir_name + "/color", images);

On Tue, Dec 5, 2017 at 4:36 PM, yan deng dengyaner@gmail.com wrote:

I have input that and the variable Dir correctly shows that also.

On Tue, Dec 5, 2017 at 4:32 PM Mihai notifications@github.com wrote:

Oh wait. That would be because you're not providing the path as an argument to the program, I imagine. you're supposed to run it using ./dynamicfusion path/to/data.

Which reminds me I haven't documented that yet

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-349488989, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KC6P0SZ3rOc7rjPR7x4jyisxA82gLks5s9eCagaJpZM4QlCbp .

mihaibujanca commented 6 years ago

You could try hardcoding the path into the code for a test.

One reason why it doesn't currently work might be different directory separators for windows paths - admittedly I should have used boost here

mihaibujanca commented 6 years ago

@mxgbs Oh I just noticed haha. The directory structure is:

data color file.color.png depth file.depth.png

This is the dataset itself though

mxgbs commented 6 years ago

Yes. I divided that to sub-folder, it works now. Mixed that two would not work even with guidance of ".color" and ".depth". Have no idea why. But new errors come after this. I'm looking for more details.

Sincerely,

Yan

On Tue, Dec 5, 2017 at 4:45 PM, Mihai notifications@github.com wrote:

@mxgbs https://github.com/mxgbs Oh I just noticed haha. The directory structure is:

data color file.color.png depth file.depth.png

This is the dataset itself though

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-349491343, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KC8d8aTSIb5bpNZyAT6AFRaz9Ybpsks5s9eOggaJpZM4QlCbp .

mihaibujanca commented 6 years ago

Well, .depth and .color is not correct, strictly speaking. Both should be .png since that is the final extension. .depth doesn't match to *depth.png

mxgbs commented 6 years ago

OK. I didn't make the the extension shown and that makes sense.

Sincerely,

Yan

On Tue, Dec 5, 2017 at 4:54 PM, Mihai notifications@github.com wrote:

Well, .depth and .color is not correct, strictly speaking. Both should be .png since that is the final extension. .depth doesn't match to *depth.png

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mihaibujanca/dynamicfusion/issues/33#issuecomment-349493150, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0KC6cRsn4A1qOAFUIZwXDId0ivm8fLks5s9eXdgaJpZM4QlCbp .