Ask Your Question

goal status in Callback in actionserver in C++

asked 2012-05-21 04:18:19 -0500

safzam gravatar image

updated 2016-11-30 21:57:19 -0500

130s gravatar image

Hi all, My client sends the goal but gets status ABORTED instead of SUCCEEDED and server gives following error:

[ WARN] [1337607518.317686823]: Your executeCallback did not set the goal to a terminal status. This is a bug in your ActionServer implementation. Fix your code! For now, the ActionServer will set this goal to aborted

My callback is

void startnode(const my_msgs::myGoalConstPtr& goal, Server* as)
  my_msgs::myResult result_;
  result_.result = 1;

Can anyone help me fix this problem?? Thanks in advance. Regards

edit retag flag offensive close merge delete



Hi, I know this is an old post. But i have your same problem. Did you solve it?? I have been trying to solve by myself for a week with no dice!!! Could you help me??? Thanks

GioRos gravatar image GioRos  ( 2014-07-31 08:58:09 -0500 )edit

Hi, I am having the same issue, do you have a lead?

Yael Green gravatar image Yael Green  ( 2016-03-16 10:48:19 -0500 )edit

This unanswered thread was closed due to 'off-topic or outdated', but I find the same error can still happen today and no other threads discuss this error. So I reopened it.

130s gravatar image 130s  ( 2016-12-01 18:47:42 -0500 )edit

3 Answers

Sort by ยป oldest newest most voted

answered 2016-12-01 18:56:31 -0500

130s gravatar image

updated 2018-07-11 06:03:54 -0500

As pointed out by the error message, fix the termination state in your Action Server. In rospy sample, not calling set_succeeded can lead to the error in this question. OP asks for C++ but I haven't tried, so I can only assume that the equivalent is setSucceeded.

edit flag offensive delete link more

answered 2019-08-17 12:38:02 -0500

I to got this error when I didn't use the function .set_preempted() in .is_preempt_requested() function. I can see that your code have no function during preempt is requested(i.e., goal is cancelled) and I hope this might be the error. my error got resolved, when I used .set_preemted() function in server code whenever preemted. and it resolved my error. hope this helps you. Thanks.

edit flag offensive delete link more

answered 2018-10-17 12:12:23 -0500

RDaneelOlivaw gravatar image

Hi here I leave a simple example that you can run in any robot with an odom topic that it might clear up some doubts on how this works. Basically that error appears when the result is not correctly set up in the setSucceeded method.

Here you have the GIT

Here you have the ROSJECT

Here you have a Video explaining how to launch it and the basic ideas behind this: VIDEO

And here the main elements of code for those that preffer having it here in the Forum, hope it helps:


#include <ros/ros.h>
#include <actionlib/server/simple_action_server.h>
#include <std_msgs/Empty.h>
#include <nav_msgs/Odometry.h> 
#include <math.h>
#include <action_status_cpp_qanda/ActOdomAction.h>

class ActOdomTurtle {

    int hz;
    float time_to_make_distance; //duration
    bool action_active;
    float x;
    float y;
    bool finish;
    float distance_to_make;
    float hypt;

    ros::NodeHandle nh;
    nav_msgs::Odometry *odom_hold; // I need this shi to hold my nav_msgs/Odom I DONT THINK I DID THIS RIGHT
    ros::Subscriber sub_read;
    ros::Rate *rate_;
    actionlib::SimpleActionServer<action_status_cpp_qanda::ActOdomAction> as_; // i might wanna rename this later
    std::string action_name_; // i need to input this for the action name;
    action_status_cpp_qanda::ActOdomResult result_; //basically in the msg these r the thing im returning

    nav_msgs::Odometry latest_odometry;

    ActOdomTurtle(std::string name):
        as_(nh, name, boost::bind(&ActOdomTurtle::executeCB, this, _1), false), 
        ROS_INFO("Inside ActOdomTurtle Action Init...");

        this->hz = 20;
        this->distance_to_make = 3;
        this->time_to_make_distance = 100;

        this->action_active = false;
        this->finish = false;

        rate_= new ros::Rate(this->hz);

        sub_read = nh.subscribe<nav_msgs::Odometry>("/odom",1,&ActOdomTurtle::giveOdomMod,this);


        ROS_INFO("Inside ActOdomTurtle Action Init...END");


    void executeCB(const action_status_cpp_qanda::ActOdomGoalConstPtr &gg){

        ROS_INFO("Action has been called...Time Starts Counting to get out of the maze");
        this->action_active = true;

        while (this->time_to_make_distance > 0 && this->finish == false ){ 

            if (as_.isPreemptRequested() || !ros::ok()){         
                ROS_INFO("%s: Preempted", action_name_.c_str());

            this->finish = this->are_we_out_of_maze();

            time_to_make_distance -= 1/(float)hz; 
            ROS_INFO("the time left is %f", time_to_make_distance);

        // If we finished because we got out of the maze, then its a success.
        if(this->finish) { 
            ROS_INFO("%s: Succeeded", action_name_.c_str());
            ROS_INFO("%s: Aborted", action_name_.c_str());
            //set the action state to aborted

        this->action_active = false;

    void giveOdomMod(const nav_msgs::Odometry::ConstPtr &_msg){ 
         * We Fetch the latest odometry data and save it in a vector 
         * **/

        this->latest_odometry.header = _msg->header;
        this->latest_odometry.child_frame_id = _msg->child_frame_id;
        this->latest_odometry.pose = _msg->pose;

        if (this->action_active)


    bool are_we_out_of_maze()
        bool result = false;

        // This is a security mesure to guarantee that the saved element in the vector is 
        // what we use for calculating.
        if (!this->result_.result_odom_array.empty())
            nav_msgs::Odometry last_odom = this->result_.result_odom_array.back();
            float last_x = last_odom.pose.pose.position.x;
            float last_y = last_odom.pose.pose.position.y;

            float hypt = sqrt(pow(last_x,2)+pow(last_y,2));
            // Because the robot starts at the origin we dont have to save the init position
            // To then compare with the latest.
            if (hypt >= this->distance_to_make ){
                ROS_WARN("SUCCESS Distance ...
edit flag offensive delete link more

Your Answer

Please start posting anonymously - your entry will be published after you log in or create a new account.

Add Answer

Question Tools



Asked: 2012-05-21 04:18:19 -0500

Seen: 3,745 times

Last updated: Oct 17 '18