search

RVMI / skiros2

A skill-based platform for ROS v.2
Other
144 stars 19 forks source link

Fix: Check length of task before access #66

Open matthias-mayr opened 1 year ago

matthias-mayr commented 1 year ago

There are circumstances under which the tasks list is empty, but will be accessed. E.g. when restarting the skill manager, but not the GUI.

frvd commented 1 year ago

So this will throw an indexerror now, right? Maybe it should be catched and throw a more meaningful error

frvd commented 1 year ago

Sorry, i misread the commit. What does it mean to return -1 instead of a task ? It seems logically wrong