From: Joseph Coffland <joseph@cauldrondevelopment.com>
Date: Sun, 2 Dec 2018 09:28:53 +0000 (-0800)
Subject: Write version to log file, Write time to log file periodically, Trigger Comm.flush... 
X-Git-Url: https://git.buildbotics.com/?a=commitdiff_plain;h=88a958e11a0d3eec651c25ad68fe15084665d814;p=bbctrl-firmware

Write version to log file, Write time to log file periodically, Trigger Comm.flush() periodically
---

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 0549a7f..db5c9c1 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,6 +1,10 @@
 Buildbotics CNC Controller Firmware Changelog
 ==============================================
 
+## v0.4.3
+ - Write version to log file.
+ - Write time to log file periodically.
+
 ## v0.4.2
  - Suppress ``Auto-creating missing tool`` warning.
  - Prevent ``Stream is closed`` error.
diff --git a/package-lock.json b/package-lock.json
index 01ec522..0ad05b1 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -1,6 +1,6 @@
 {
   "name": "bbctrl",
-  "version": "0.4.2",
+  "version": "0.4.3",
   "lockfileVersion": 1,
   "requires": true,
   "dependencies": {
diff --git a/package.json b/package.json
index 6f3701d..8f627c5 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "bbctrl",
-  "version": "0.4.2",
+  "version": "0.4.3",
   "homepage": "http://buildbotics.com/",
   "repository": "https://github.com/buildbotics/bbctrl-firmware",
   "license": "GPL-3.0+",
diff --git a/src/py/bbctrl/Mach.py b/src/py/bbctrl/Mach.py
index ac38a79..7e7ba5d 100644
--- a/src/py/bbctrl/Mach.py
+++ b/src/py/bbctrl/Mach.py
@@ -83,7 +83,7 @@ class Mach(Comm):
         self.last_cycle = 'idle'
 
         ctrl.state.set('cycle', 'idle')
-        self._update_cycle_cb(False)
+        self._update_cb(False)
 
         ctrl.state.add_listener(self._update)
 
@@ -118,9 +118,12 @@ class Mach(Comm):
                             (cycle, current))
 
 
-    def _update_cycle_cb(self, now = True):
-        if now: self._update_cycle()
-        self.ctrl.ioloop.call_later(1, self._update_cycle_cb)
+    def _update_cb(self, now = True):
+        if now:
+            self._update_cycle()
+            self.flush()
+
+        self.ctrl.ioloop.call_later(1, self._update_cb)
 
 
     def _update_cycle(self):
diff --git a/src/py/bbctrl/__init__.py b/src/py/bbctrl/__init__.py
index b169d97..9486017 100644
--- a/src/py/bbctrl/__init__.py
+++ b/src/py/bbctrl/__init__.py
@@ -34,6 +34,7 @@ import tornado
 import argparse
 import logging
 import datetime
+import pkg_resources
 
 from pkg_resources import Requirement, resource_filename
 
@@ -80,6 +81,11 @@ def on_exit(sig = 0, func = None):
     sys.exit(1)
 
 
+def log_time(log, ioloop):
+    log.info(datetime.datetime.now().strftime('%Y/%m/%d %H:%M:%S'))
+    ioloop.call_later(60 * 60, log_time, log, ioloop)
+
+
 def parse_args():
     parser = argparse.ArgumentParser(
         description = 'Buildbotics Machine Controller')
@@ -139,8 +145,8 @@ def run():
         root.addHandler(h)
 
     # Log header
-    now = datetime.datetime.now().strftime('%Y/%m/%d %H:%M:%S')
-    root.info('Log started ' + now)
+    version = pkg_resources.require('bbctrl')[0].version
+    root.info('Log started v%s' % version)
 
     # Set signal handler
     signal.signal(signal.SIGTERM, on_exit)
@@ -148,6 +154,9 @@ def run():
     # Create ioloop
     ioloop = tornado.ioloop.IOLoop.current()
 
+    # Write time to log periodically
+    log_time(root, ioloop)
+
     # Start controller
     ctrl = Ctrl(args, ioloop)