ori-drs / pronto_anymal_example

Example binary running the Pronto state estimator on the ANYmal B robot
24 stars 10 forks source link

changes to cmakelists.txt in robocongen pkg. #6

Closed shivauchiha closed 4 years ago

shivauchiha commented 4 years ago

Hi! so when i tried to build this package in my catkin_ws , I got a error stating that it was not able to find transforms.cpp and the other allied files present in robocogen source folder . So I just changed the address for 7 files in cmakelists.txt of robocogen pkg to place where the files are present .

mcamurri commented 4 years ago

Thank you @shivauchiha for your contribution. It seems that catkin build and catkin_make fill the variable CMAKE_SOURCE_DIR differently. I suspect that your fix would make catkin_make compile but thencatkin build would not work.

Can you please try to change to the following and see if it works with catkin_make ?

set(SOURCES src/transforms.cpp
                         src/jacobians.cpp
                         src/jsim.cpp
                         src/inverse_dynamics.cpp
                         src/forward_dynamics.cpp
                         src/inertia_properties.cpp
                         src/miscellaneous.cpp)

I think the relative path should work for both systems.

shivauchiha commented 4 years ago

Hi @mcamurri ! Thanks for the response . Yes the above changes to cmakefile.txt as you have suggested works for both catkin_make and catkin build . The demonstration runs successfully in both cases with only one caveat in catkin_make case. Even though the demo runs perfectly there is warning "too many lvls of symbolic links error" similar to one seen in issue #5

I think for now catkin build is better way to get the demo running.

shivauchiha commented 4 years ago

@mcamurri I am closing this request as the repo works as intended and no changes was needed . Thank you for your quick response and help !