Add MoveIt Task Constructor packages (core, capabilities, visualization, demo, rviz_marker_tools)#14
Conversation
|
Thanks @iamnambiar! There is some issue on Windows, please take a look and let us know if you need help. Best, Tobi |
|
@Tobias-Fischer Line 160 in ff42cf0 I'm hoping this same issue came up when someone previously tried to build it for Windows. I'm happy to fix this as part of this PR by adding a patch. However, since the package has already been added, this would require incrementing the build number to generate a new recipe, which would also regenerate recipes for all the other packages. I'm not sure if that's the way to do this. Or else, I can update the vinca.yaml file to disable and skip the Windows build for now. What do you think is the best approach here? |
Updated the import statements to use 'module_' instead of 'module' for C++20 compatibility.
Updated CMakeLists.txt and header files for C++20 compatibility by renaming module and exporting functions correctly.
|
I fixed the py_binding_tools issue. There are now other Windows errors, can you please have a go at fixing them @iamnambiar |
|
@Tobias-Fischer Thank you for fixing the py_binding_tools issue. I was actually running this locally on my windows partition now and trying to fix the issues one by one. |
|
Cool I'll let you do, thanks :) |
This PR adds the six MoveIt Task Constructor packages to ros-lyrical:
along with patches needed to build
moveit-task-constructor-coreandmoveit-task-constructor-visualization.