lnotspotl / tbai

Towards better athletic intelligence
49 stars 11 forks source link

fixed the tbai_rl package build issue #11

Closed rua0ra1 closed 1 month ago

rua0ra1 commented 1 month ago

Hi @lnotspotl , I followed the given build instructions. I had following problem while build tbai_rl package as shown in figure.

issue_with_grid_map

I fixed the issue in this pull request. I think the main issue is with building tbai_rl, the build system can't access the headers of the grid_map_ros and grid_map_core while building this package.

I fixed this problem by changing the tbai_gridmap package cmakelist file from

catkin_package(
  INCLUDE_DIRS include
  LIBRARIES tbai_gridmap 
 CATKIN_DEPENDS  
 DEPENDS OpenCV
)

to

catkin_package(
  INCLUDE_DIRS include
  LIBRARIES tbai_gridmap 
 CATKIN_DEPENDS  grid_map_ros grid_map_core
 DEPENDS OpenCV
)

let me know if you need any further information from my side. Thank you

lnotspotl commented 1 month ago

LGTM. Thanks for the fix :)

rua0ra1 commented 1 month ago

All good. I am glad to contribute :)