OrebroUniversity / hiqp

GNU General Public License v3.0
9 stars 2 forks source link

Comprehensive list of services #19

Closed neckutrek closed 7 years ago

neckutrek commented 7 years ago

Could a comprehensive list of services for hiqp be:

Regarding implementation, all services should be kept in its own class with a pure-virtual interface for implementing the effect of such calls. A ROS/HiQP controller would then inherit from both BaseController and this new HiQPServices class/interface. Or HiQPServices gets a pointer to task manager and sends all service calls to the manager.

rtkg commented 7 years ago

Looks good, one question: with (de)activating/(de)monitoring priority levels you mean (de)activating/(de)monitoring all tasks at a given priority, right?

neckutrek commented 7 years ago

Yes

Le 17 déc. 2016 13:38, "Robert Krug" notifications@github.com a écrit :

Looks good, one question: with (de)activating/(de)monitoring priority levels you mean (de)activating/(de)monitoring all tasks at a given priority, right?

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/neckutrek/hiqp/issues/19#issuecomment-267760606, or mute the thread https://github.com/notifications/unsubscribe-auth/ADmjhRs47fF_cQfsHk7M7sZGZsS-kLBJks5rI9fagaJpZM4LPE6q .

neckutrek commented 7 years ago

The service add_primitive has changed name to set_primitive.

neckutrek commented 7 years ago

All have been implemented except for remove_primitives_with_priority (which doesn't make sense as all those primitives have dependencies to tasks), and list_priority_level (which is achievable instead by using list_all_primitives and list_all_tasks). Latest on devel.