Merge pull request #37 from fmessmer/fix/exception_logging

Fix/exception logging
mojin-robotics · May 16, 2024 · 797ae6b · 797ae6b
2 parents 91921dd + 55f4402
commit 797ae6b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/flexbe_core/src/flexbe_core/logger.py b/flexbe_core/src/flexbe_core/logger.py
@@ -42,8 +42,8 @@ def log_throttle(period, text, severity):
         # only log when it's the first time or period time has passed for the logging message
         if not log_id in Logger._last_logged.keys() or \
             rospy.Time.now().to_sec() - Logger._last_logged[log_id].to_sec() > period:
-                Logger.log(text, severity)
-                Logger._last_logged.update({log_id: rospy.Time.now()})
+            Logger.log(text, severity)
+            Logger._last_logged.update({log_id: rospy.Time.now()})
 
     @staticmethod
     def local(text, severity):

diff --git a/flexbe_onboard/src/flexbe_onboard/flexbe_onboard.py b/flexbe_onboard/src/flexbe_onboard/flexbe_onboard.py
@@ -153,7 +153,7 @@ def _behavior_execution(self, msg):
                 self._pub.publish(self.status_topic, BEStatus(behavior_id=msg.behavior_checksum, code=BEStatus.FAILED))
                 Logger.logerr('Behavior execution failed!\n%s' % str(e))
                 import traceback
-                Logger.loginfo(traceback.format_exc())
+                Logger.loginfo('Traceback:\n' % str(traceback.format_exc()))
                 result = result if result is not None else "exception"  # only set result if not executed
 
             # done, remove left-overs like the temporary behavior file