Removed automatic update of skill details

source/RobotAPI/libraries/skills_gui/SkillMemoryGui.cpp

@@ -107,8 +107,12 @@ namespace armarx::skills::gui
 
         // update cascade
         connect(this, &SkillMemoryGUI::updateGui, skillGroupBox, &SkillGroupBox::updateGui);
+
+        // we don't want to update the skill details periodically, update can be triggered manually
+        /*
         connect(
             this, &SkillMemoryGUI::updateGui, skillDetailGroupBox, &SkillDetailGroupBox::updateGui);
+        */
         connect(this,
                 &SkillMemoryGUI::updateGui,
                 skillExecutionTreeWidget,

source/RobotAPI/libraries/skills_gui/skill_details/SkillDetailsTreeWidget.cpp

@@ -34,8 +34,11 @@ namespace armarx::skills::gui
     SkillDetailsTreeWidget::updateContents(const SkillID& skillId, const SkillDescription& descr)
     {
         // dont touch the widget if the skill id didn't change
+        // note: this is only relevant when periodic updates are enabled
+        /*
         if (shownSkill.has_value() && skillId == shownSkill.value().skillId)
             return;
+        */
 
         // check the parameters: did they change?
         if (shownSkill.has_value())
@@ -47,7 +50,7 @@ namespace armarx::skills::gui
                 // for now, we just overwrite without asking... (and do nothing)
             }
             // other cases: if the parameter types change, we *have* to reset; else the skill
-            // (probably) can't be started with the parameters.
+            // cannot be started with the parameters.
             // same goes for the result type.
         }
 

source/RobotAPI/libraries/skills_gui/skills/SkillTreeWidget.cpp

@@ -20,12 +20,6 @@ namespace armarx::skills::gui
             return;
         }
 
-        if (selectedSkill.skillId == c->skillId)
-        {
-            // no change
-            return;
-        }
-
         selectedSkill.skillId = c->skillId;
         emit updateSkillDetails(selectedSkill.skillId);
     }