search

squirrel-project / squirrel_planning

Repository for planning related SQUIRREL packages.
0 stars 8 forks source link

(KCL) Integration meeting update #10

Closed m312z closed 9 years ago

m312z commented 9 years ago

Included an updated domain for the push task. Added the move_base interface for PDDL->move_base translation. Set up launch files for knowledge only, planning only, or everything.

ipa-jsf commented 9 years ago

@m312z Did you updated your repository with the main branch? I had to resolve merge conflicts, which shouldn't happen if you did so. Please do so next time.

Before accepting the PR, please send me an updated PR for squirrel_planning_knowledge_msgs, as I get the following error when compiling:

error: ‘DOMAIN_FUNCTION’ is not a member of ‘squirrel_planning_knowledge_msgs::KnowledgeItem’
if(attr.knowledge_type == squirrel_planning_knowledge_msgs::KnowledgeItem::DOMAIN_FUNCTION && attr.attribute_name.compare(ait->first)==0)

There are also some catkin_lint errors (catkin_lint src/) that should be fixed:

squirrel_planning_system: error: unconfigured build_depend on 'geometry_msgs'
squirrel_planning_system: error: unconfigured build_depend on 'diagnostic_msgs'
ipa-jsf commented 9 years ago

PR accepted :100: