changed warning to debug/info and reduced spam if no maxHistorySize is set

source/RobotAPI/libraries/armem/server/segment/SpecializedCoreSegment.h

@@ -27,7 +27,7 @@ namespace armarx::armem::server::segment
                 MemoryToIceAdapter& iceMemory,
                 const std::string& defaultCoreSegmentName = "",
                 aron::type::ObjectPtr coreSegmentAronType = nullptr,
-                int defaultMaxHistorySize = -1,
+                int defaultMaxHistorySize = 10,
                 const std::vector<PredictionEngine>& predictionEngines = {});
 
             virtual ~SpecializedCoreSegment() override;

source/RobotAPI/libraries/armem/server/segment/SpecializedProviderSegment.h

@@ -29,7 +29,7 @@ namespace armarx::armem::server::segment
                 const std::string& defaultCoreSegmentName = "",
                 aron::type::ObjectPtr providerSegmentAronType = nullptr,
                 aron::type::ObjectPtr coreSegmentAronType = nullptr,
-                int defaultMaxHistorySize = -1,
+                int defaultMaxHistorySize = 10,
                 const std::vector<PredictionEngine>& providerSegmentPredictionEngines = {},
                 const std::vector<PredictionEngine>& coreSegmentPredictionEngines = {});
 

source/RobotAPI/libraries/armem/server/wm/detail/MaxHistorySize.cpp

@@ -5,11 +5,21 @@
 
 namespace armarx::armem::server::wm::detail
 {
-    void MaxHistorySize::setMaxHistorySize(long maxSize)
+    void MaxHistorySize::setMaxHistorySize(long maxSize, const std::string& additionalInfo)
     {
         if (maxSize < 0){
-            ARMARX_WARNING << "The maxHistorySize for this core segment is set to < 0. This means nothing will ever be forgotten in working memory. "
-                           << "This may slow down the memory server.";
+            if(additionalInfo != ""){
+                ARMARX_INFO << "The maxHistorySize for this memory entity is set to < 0. "
+                         << "This means nothing will ever be forgotten in working memory. "
+                         << "This may slow down the memory server."
+                         << "It is better to set the maxHistorySize per Provider- or CoreSegment";
+            } else {
+                
+                ARMARX_DEBUG << "The maxHistorySize for this memory entity is set to < 0. "
+                            << "This means nothing will ever be forgotten in working memory. "
+                            << "This may slow down the memory server."
+                            << "The entity name is " << additionalInfo;
+            }
         }
         this->_maxHistorySize = maxSize;
     }

source/RobotAPI/libraries/armem/server/wm/detail/MaxHistorySize.h

@@ -12,7 +12,7 @@ namespace armarx::armem::server::wm::detail
          * @brief Set the maximum number of snapshots to be contained in an entity.
          * Affected entities are to be update right away.
          */
-        void setMaxHistorySize(long maxSize);
+        void setMaxHistorySize(long maxSize, const std::string& additionalInfo = "");
 
         long getMaxHistorySize() const;
 

source/RobotAPI/libraries/armem/server/wm/memory_definitions.cpp

@@ -16,7 +16,8 @@ namespace armarx::armem::server::wm
 
     void Entity::setMaxHistorySize(long maxSize)
     {
-        MaxHistorySize::setMaxHistorySize(maxSize);
+        std::string containerID = (this->id()).str();
+        MaxHistorySize::setMaxHistorySize(maxSize, containerID);
         truncate();
     }
 

source/RobotAPI/libraries/armem/server/wm/memory_definitions.h

@@ -101,7 +101,15 @@ namespace armarx::armem::server::wm
         ProviderSegment& addProviderSegment(const std::string& name, Args... args)
         {
             ProviderSegmentT& added = CoreSegmentBase::addProviderSegment(name, args...);
-            added.setMaxHistorySize(this->getMaxHistorySize());
+            int maxHistorySize = this->getMaxHistorySize();
+            if(maxHistorySize < 0){
+                ARMARX_INFO << "The maxHistorySize for this core segment is set to < 0. "
+                            << "This means nothing will ever be forgotten in working memory. "
+                            << "This may slow down the memory server. \n"
+                            << "Core Segment Name: "
+                            << (this->id()).str();
+            }
+            added.setMaxHistorySize(maxHistorySize);
             return added;
         }