sbillin / IMLP

BSD 3-Clause "New" or "Revised" License
18 stars 8 forks source link

Issues while building ICP_App #2

Open supritimulay opened 6 years ago

supritimulay commented 6 years ago

Hi,

While Building the ICP_App in VS2015 I am getting linker error cisstICP.lib(utilities.obj) : error LNK2019: unresolved external symbol "public: cdecl Wm5::NoniterativeEigen3x3::NoniterativeEigen3x3(class Wm5::Matrix3 const &)" (??0?$NoniterativeEigen3x3@N@Wm5@@QEAA@AEBV?$Matrix3@N@1@@Z) referenced in function "void cdecl ComputeCovEigenDecomposition_NonIter(class vctFixedSizeMatrix<double,3,3,1> const &,class vctFixedSizeVector<double,3> &,class vctFixedSizeMatrix<double,3,3,1> &)" (?ComputeCovEigenDecomposition_NonIter@@YAXAEBV?$vctFixedSizeMatrix@N$02$02$00@@AEAV?$vctFixedSizeVector@N$02@@AEAV1@@Z).

In additional dependencies, I have added all the WildMagic libraries but still getting a linker error for ICP_App. Please advice.

Thanks

sbillin commented 3 years ago

Hi, sorry I've been off the grid on this for a while. In case others are encountering this problem, my guess is that your CMake configuration does not have the path set to find the WildMagic libraries.

After configuring CMake, check the CMake configuration values to verify that WM5_BASE_DIR and WM5_LIB_PATH are set to the proper directory. The CMakeLists.txt file provides hints for where to look but doesn't really know where to find the correct path and has to be set manually. These paths should be set to the directories that contain "wm5path.txt" and "Wm5Core.lib" files, respectively, as indicated by the "NAMES" property in CMakeLists.txt.

If you don't want to manually set these paths every time you reconfigure CMake, you can change your CMakeLists.txt to set the proper paths for your system using the "PATHS" property associated with each value. Hope that helps.

sbillin commented 3 years ago

Just posted a comment to this issue. Hope that helps

From: ZYJiang97 notifications@github.com Sent: Saturday, December 26, 2020 3:10 PM To: sbillin/IMLP IMLP@noreply.github.com Cc: Subscribed subscribed@noreply.github.com Subject: [EXT] Re: [sbillin/IMLP] Issues while building ICP_App (#2)

APL external email warning: Verify sender noreply@github.com before clicking links or attachments

Have you solved this issue? I get the same problem.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/sbillin/IMLP/issues/2#issuecomment-751391600, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACEXGE7Y7MS2HWE3ZYTCCTDSWY7IZANCNFSM4EB6G4PA.

ZYJiang97 commented 3 years ago

Yeah. I set WM5_BASE_DIR and WM5_LIB_PATH correctly. I can build cisstICP without any error. This error came from the building of ICP_App. Is this the problem of cisst library? err

sbillin commented 3 years ago

Hmm, did you set WM5_BASE_DIR and WM5_LIB_PATH in the CMake configurations for both the cisstICP build and for the ICP_App build? They would need to be set in both places. The only difference I see between the two builds is that ICP_App has an additional conditional that will require setting WM5_BASE_DIR first then reconfiguring then setting WM5_LIB_PATH and WM5_LIB_INCLUDE next and reconfiguring again.

If that all checks out I’m not sure what the issue would be. I’m not aware of an issue with cisstICP but it’s been a while since I had it running so something with a build tool, etc. could have changed in the interim. I’m sorry I don’t have a visual studio build framework setup on my system to check into it more deeply at the moment…

From: ZYJiang97 notifications@github.com Sent: Monday, December 28, 2020 9:29 AM To: sbillin/IMLP IMLP@noreply.github.com Cc: Billings, Seth D. Seth.Billings@jhuapl.edu; Comment comment@noreply.github.com Subject: [EXT] Re: [sbillin/IMLP] Issues while building ICP_App (#2)

APL external email warning: Verify sender noreply@github.com before clicking links or attachments

Yeah. I set WM5_BASE_DIR and WM5_LIB_PATH correctly. I can build cisstICP without any error. This error came from the building of ICP_App. Is this the problem of cisst library? [Image removed by sender. err]https://user-images.githubusercontent.com/71422769/103220700-f1bc5500-48ee-11eb-97d7-25983ec57cc1.png

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/sbillin/IMLP/issues/2#issuecomment-751730205, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACEXGEYEIQI6FDQU2A6JN73SXCI3PANCNFSM4EB6G4PA.

sbillin commented 3 years ago

Oh sorry, I was responding via email earlier and didn't see the screenshot that was included. So yes these errors do look like a cisst library issue, and appears to be the same errors mentioned in the README that I had encountered when compiling with Visual Stuido 14 2015 Win64. When I was developing this, I had to compile using the earlier version of Visual Studio 12 2013 Win64 in order to get it to work correctly.

If you want to try to track down the solution, you may get some help by posting an issue to the cisst library Github repo. I don't know enough about the cisst library myself to be of much help there. If I had time to work on this code base again, I think I would do a port to Eigen for the matrix computations to make the code a bit easier to setup and compile.

ZYJiang97 commented 3 years ago

Thanks a lot.