Skip to content
Snippets Groups Projects
Commit 1a46b1b2 authored by Fabian Tërnava's avatar Fabian Tërnava
Browse files

fix missing result for the last status update if a skill succeeds

parent 7b2af2e4
No related branches found
No related tags found
1 merge request!412Skills: Fix missing result for the last status update if a skill succeeds
Pipeline #16312 passed
......@@ -29,7 +29,8 @@ namespace armarx::skills::provider
.rootProfileDefaults = root_profile_params.toAron(),
.timeout = armarx::core::time::Duration::MilliSeconds(1000),
.parametersType =
armarx::skills::Example::HelloWorldAcceptedType::ToAronType()};
armarx::skills::Example::HelloWorldAcceptedType::ToAronType(),
.resultType = armarx::skills::Example::HelloWorldAcceptedType::ToAronType()};
}
Skill::MainResult
......@@ -42,6 +43,6 @@ namespace armarx::skills::provider
.dump(2)
<< "\n"
<< "(executed at: " << IceUtil::Time::now() << ")";
return {TerminatedSkillStatus::Succeeded, nullptr};
return {TerminatedSkillStatus::Succeeded, in.parameters.toAron()};
}
} // namespace armarx::skills::provider
......@@ -340,7 +340,7 @@ namespace armarx
// Exit succeeded!
// All succeeded!
{
updateStatus(SkillStatus::Succeeded);
updateStatus(SkillStatus::Succeeded, mainRet.data);
// return result of main method
std::unique_lock l(skillStatusesMutex);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment