Use forward declaration, use fromString() instead of constructor

source/RobotAPI/libraries/armem_gui/prediction_widget/PredictionWidget.cpp

@@ -30,6 +30,9 @@
 #include <QSpinBox>
 #include <QVBoxLayout>
 
+#include "TimestampInput.h"
+
+
 namespace armarx::armem::gui
 {
     PredictionWidget::PredictionWidget(GetEntityInfoFn&& entityInfoRetriever) :
@@ -101,7 +104,7 @@ namespace armarx::armem::gui
     void
     PredictionWidget::updateCurrentEntity()
     {
-        MemoryID entityID(memoryEntity->text().toStdString());
+        MemoryID entityID = MemoryID::fromString(memoryEntity->text().toStdString());
         predictionEngineSelector->clear();
         if (!entityID.hasGap() && entityID.hasEntityName())
         {
@@ -123,7 +126,7 @@ namespace armarx::armem::gui
     void
     PredictionWidget::startPrediction()
     {
-        MemoryID entityID(memoryEntity->text().toStdString());
+        MemoryID entityID = MemoryID::fromString(memoryEntity->text().toStdString());
         armarx::DateTime timestamp;
         for (const auto& [inputKey, input] : timestampInputs)
         {

source/RobotAPI/libraries/armem_gui/prediction_widget/PredictionWidget.h

@@ -31,7 +31,6 @@
 #include <RobotAPI/libraries/armem/core/Prediction.h>
 #include <RobotAPI/libraries/aron/core/type/variant/forward_declarations.h>
 
-#include "TimestampInput.h"
 
 class QComboBox;
 class QHBoxLayout;
@@ -39,14 +38,17 @@ class QLineEdit;
 class QPushButton;
 class QSpinBox;
 
+
 namespace armarx::armem::gui
 {
+    class TimestampInput;
+
+
     class PredictionWidget : public QWidget
     {
         Q_OBJECT // NOLINT
 
     public:
-
         struct EntityInfo
         {
             aron::type::ObjectPtr type = nullptr;