Skip to content
Snippets Groups Projects
Verified Commit c8999b29 authored by Peter Albrecht's avatar Peter Albrecht :monkey:
Browse files

feat: history selector updating

parent 46879760
No related branches found
No related tags found
1 merge request!512Skill-Gui Stability and QOL
Pipeline #22564 failed
...@@ -428,6 +428,7 @@ namespace armarx::skills::gui ...@@ -428,6 +428,7 @@ namespace armarx::skills::gui
this->addParametersToHistory(skillId, params); this->addParametersToHistory(skillId, params);
ARMARX_INFO << "Saved skill parameters to history..."; ARMARX_INFO << "Saved skill parameters to history...";
ARMARX_IMPORTANT << "Executing skill from GUI: " << skillId << "."; ARMARX_IMPORTANT << "Executing skill from GUI: " << skillId << ".";
try try
......
#include "SkillDetailsGroupBox.h" #include "SkillDetailsGroupBox.h"
#include <qpushbutton.h> #include <qpushbutton.h>
#include "RobotAPI/libraries/skills_gui/skill_details/ProfileMenuWidget.h"
#include "RobotAPI/libraries/skills_gui/skill_details/SkillDetailsTreeWidget.h" #include "RobotAPI/libraries/skills_gui/skill_details/SkillDetailsTreeWidget.h"
namespace armarx::skills::gui namespace armarx::skills::gui
...@@ -80,6 +81,9 @@ namespace armarx::skills::gui ...@@ -80,6 +81,9 @@ namespace armarx::skills::gui
if (skillId.has_value()) if (skillId.has_value())
{ {
memory->startExecutionWithParams(skillId.value(), data); memory->startExecutionWithParams(skillId.value(), data);
// update the history selector, since it definitely has an entry now.
emit executed(skillId.value());
} }
} }
...@@ -165,6 +169,14 @@ namespace armarx::skills::gui ...@@ -165,6 +169,14 @@ namespace armarx::skills::gui
&QPushButton::pressed, &QPushButton::pressed,
skillDetailsTreeWidget, skillDetailsTreeWidget,
&SkillDetailsTreeWidget::reloadLastParameters); &SkillDetailsTreeWidget::reloadLastParameters);
connect(skillDetailsTreeWidget,
&SkillDetailsTreeWidget::updated,
profileMenuWidget,
&ProfileMenuWidget::updateHistorySelector);
connect(this,
&SkillDetailGroupBox::executed,
profileMenuWidget,
&ProfileMenuWidget::updateHistorySelector);
} }
......
...@@ -6,6 +6,7 @@ ...@@ -6,6 +6,7 @@
#include <QSplitter> #include <QSplitter>
#include <QVBoxLayout> #include <QVBoxLayout>
#include "RobotAPI/libraries/skills/core/SkillID.h"
#include "RobotAPI/libraries/skills_gui/aron_tree_widget/AronTreeWidgetController.h" #include "RobotAPI/libraries/skills_gui/aron_tree_widget/AronTreeWidgetController.h"
#include "RobotAPI/libraries/skills_gui/aron_tree_widget/widgets/SkillDescriptionWidget.h" #include "RobotAPI/libraries/skills_gui/aron_tree_widget/widgets/SkillDescriptionWidget.h"
...@@ -34,6 +35,7 @@ namespace armarx::skills::gui ...@@ -34,6 +35,7 @@ namespace armarx::skills::gui
void disconnectGui(); void disconnectGui();
void updateGui(SkillManagerWrapper::Snapshot update); void updateGui(SkillManagerWrapper::Snapshot update);
void executed(const skills::SkillID skillId);
public slots: public slots:
/** /**
......
...@@ -227,6 +227,7 @@ namespace armarx::skills::gui ...@@ -227,6 +227,7 @@ namespace armarx::skills::gui
return; return;
} }
ARMARX_INFO << "Reloaded parameters from the last execution";
aronTreeWidgetController->setFromAron(params.value()); aronTreeWidgetController->setFromAron(params.value());
} }
......
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