farmbot-serial 0.3.2 → 0.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0af517cc7b6a0ead011b89a0a4f6273fe01edde8
-  data.tar.gz: ab3b7f12f5dc949d03021c1a63aa6ca0896b336d
+  metadata.gz: 348ccc82a174231787613518c05ef3da32c544c7
+  data.tar.gz: 772b3ea17280788b8cfd1ad1fbb5f3f1747d9cd3
 SHA512:
-  metadata.gz: b7b9a28154e94143155b47e0a548005e9b0744cc036c7159a658126f4e778ea7fe7a2830e0262df4f918e0becdb50ca7abc7087540bae1cd7eb8133f10ff3ffd
-  data.tar.gz: 95905ed4b8e95d57ae11fd0d90774c2e31b07019735f9342e3d16c1f505101ba197f96460e6856f7ddca62793b39dcd48a06f333657ab6635bde9ce84fec21d8
+  metadata.gz: f2640d61eae76ba0e5088698143aed20c8b7e870d997d2153009602476373eb053380d3f0a69ab5fc0d6c1284166d077a63ac3971d535e2ab7eb865a7bed6ed6
+  data.tar.gz: 253323c3ec8f39795bfaa965d14d8f715fb0a7294408504cfd9aa4d0e7f315c324d7a4fd1ea9f1b7755aee4edb49da645f7c21f77fc580c3bafc67b202f4dbeb

data/console.rb

@@ -50,7 +50,6 @@ EM.run do
   end
   bot.onchange  { |diff|  puts "STATUS CHANGE: #{diff};" }
   bot.onclose { puts "bye!"; EM.stop } # Unplug the bot and see
-  # EventMachine::PeriodicTimer.new(7) { print '.'; bot.serial_port.puts "F31 P8" }
   EM.open_keyboard(KeyboardHandler, bot)
   0.upto(14) { |num| bot.commands.read_parameter(num) }
 

data/lib/arduino.rb

@@ -36,7 +36,6 @@ module FB
     # Send outgoing test to arduino from pi
     def write(gcode)
       @outbound_queue.unshift gcode
-      execute_command_next_tick
     end
 
     def onchange(&blk)
@@ -62,10 +61,8 @@ module FB
       Position.new(status[:X], status[:Y], status[:Z])
     end
 
-    def execute_command_next_tick
-      EM.next_tick do
-        status.ready? ? pop_gcode_off_queue : execute_command_next_tick
-      end
+    def maybe_execute_command
+      pop_gcode_off_queue if status.ready?
     end
 
     def next_cmd
@@ -85,6 +82,7 @@ module FB
     end
 
     def start_event_listeners
+      EM.tick_loop { maybe_execute_command } # A noble experiment.
       status.onchange { |diff| @onchange.call(diff) if @onchange }
       inbound_queue.subscribe do |gcodes|
         Array(gcodes).each do |gcode|

data/spec/spec_helper.rb

@@ -3,6 +3,7 @@ require 'simplecov'
 SimpleCov.start do
   add_filter "/spec/"
 end
+
 require 'pry'
 require 'farmbot-serial'
 
@@ -10,7 +11,20 @@ require_relative 'fakes/fake_serial_port'
 require_relative 'fakes/fake_logger'
 require_relative 'fakes/fake_arduino'
 
+require 'ruby-prof'
+
 RSpec.configure do |config|
+  config.before(:suite) do
+    # Profile the code
+    RubyProf.start
+  end
+
+  config.after(:suite) do
+    result = RubyProf.stop
+    # Print a flat profile to text
+    printer = RubyProf::FlatPrinter.new(result)
+    printer.print(STDOUT)
+  end
 end
 
 # This is used for testing things that require an event loop. Once run, you can

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: farmbot-serial
 version: !ruby/object:Gem::Version
-  version: 0.3.2
+  version: 0.3.3
 platform: ruby
 authors:
 - Tim Evers
@@ -81,6 +81,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: ruby-prof
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: eventmachine
   requirement: !ruby/object:Gem::Requirement