SkillManagerGUI: Constructing new skill instances each time the "Update and Search" button is pressed
Summary: Constructors of skills are called very often, which might be problematic.
Current behavior and steps to reproduce: Write an ARMARX_INFO, for example, in the constructor of the "Say" skill at armarx/skills/speech source/armarx/speech/skills/skills/Say.cpp
. Start, for example in simulation, all necessary dependencies and open a SkillManagerGUI. Each time the Update and Search
button is clicked, the message will again appear in the log. Due to de-duplication, make sure to have a look at the original log in the terminal rather than the one in the log viewer.
Expected behavior: A skill is only constructed when it is invoked, plus potentially once in the beginning to have an instance already available.