sonamp 0.0.5 → 0.0.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 816d27d5e7d9e1e5c1ab289d18dce3fdddb18a8fa15bb19ca51f776adfe0fdd0
-  data.tar.gz: faffd6868c0c625f8a7ae839cbc00a53f9d1d8d75124c00691c035f18c1af8e5
+  metadata.gz: 6a0be380a5a03184319da1aa8806ebe909a214107990470529bc550aaab6b207
+  data.tar.gz: 22835f026d6ee858d59fd5ecce4fe31b73b4e5c21b709ed204c228a6939618fd
 SHA512:
-  metadata.gz: 1335f8bf639400ad932b0672919160b4754c41dbd46dbcd7107145d8147a648b4f29705a3eff322ffb39c21d5c13229e6f622c2885c5a792ad9aba8420566bfe
-  data.tar.gz: 8cbf78ea9632fdefa9d50270890396c3722bbb94654f003fad5746c4e341456f4c66c4cd6d4f5fd7202c1f59dd951afef96aa201079388d8ae818d457c7de63e
+  metadata.gz: aedbed354c3bce410be5b6911ffc7f586611901f5322a61fccce527bec01478d374b9921d56600c4ae3a884225babf74866bf019d265e8b1778fbb75292f004f
+  data.tar.gz: 212576c917c388143d8db98526f73b6acf6b8340a8ebd56b16026cf415fa4b8a979bbabf3e6ba29e49a4a30f09ea262c3cc6a18a517cb52891484aa17bcde66d

data/.gitignore

@@ -0,0 +1 @@
+*.gem

data/lib/sonamp/client.rb

@@ -1,15 +1,51 @@
+require 'timeout'
+
 module Sonamp
   class Error < StandardError; end
   class InvalidCommand < Error; end
   class NotApplicable < Error; end
   class UnexpectedResponse < Error; end
+  class CommunicationTimeout < Error; end
+
+  RS232_TIMEOUT = 3
+  DEFAULT_DEVICE_GLOB = '/dev/ttyUSB*'
+
+  module_function def detect_device(*patterns, logger: nil)
+    if patterns.empty?
+      patterns = [DEFAULT_DEVICE_GLOB]
+    end
+    devices = patterns.map do |pattern|
+      Dir.glob(pattern)
+    end.flatten.uniq
+    queue = Queue.new
+    threads = devices.map do |device|
+      Thread.new do
+        Timeout.timeout(RS232_TIMEOUT * 2) do
+          logger&.debug("Trying #{device}")
+          Client.new(device, logger: logger).get_zone_power
+          logger&.debug("Found amplifier at #{device}")
+          queue << device
+        end
+      rescue CommunicationTimeout, IOError, SystemCallError => exc
+        logger&.debug("Failed on #{device}: #{exc.class}: #{exc}")
+      end
+    end
+    wait_thread = Thread.new do
+      threads.map(&:join)
+      queue << nil
+    end
+    queue.shift.tap do
+      threads.map(&:kill)
+      wait_thread.kill
+    end
+  end
 
   class Client
     def initialize(device = nil, logger: nil)
       @logger = logger
 
       if device.nil?
-        device = Dir['/dev/ttyUSB*'].sort.first
+        device = Sonamp.detect_device(logger: logger)
         if device
           logger&.info("Using #{device} as TTY device")
         end
@@ -38,29 +74,25 @@ module Sonamp
       dispatch_assert(cmd, expected)
     end
 
-    def get_zone_volume(zone = nil)
-      if zone
-        if zone < 1 || zone > 4
-          raise ArgumentError, "Zone must be between 1 and 4: #{zone}"
-        end
-        Integer(dispatch_extract_suffix(":V#{zone}?", 'V'))
-      else
-        dispatch(":VG?", 4).map do |resp|
-          Integer(extract_suffix(resp, 'V'))
-        end
+    def power_off
+      1.upto(4).each do |zone|
+        set_zone_power(zone, false)
       end
     end
 
+    def get_zone_volume(zone = nil)
+      get_zone_value('V', zone)
+    end
+
     def set_zone_volume(zone, volume)
-      if zone < 1 || zone > 4
-        raise ArgumentError, "Zone must be between 1 and 4: #{zone}"
-      end
       if volume < 0 || volume > 100
         raise ArgumentError, "Volume must be between 0 and 100: #{volume}"
       end
-      cmd = ":V#{zone}#{volume}"
-      expected = cmd[1...cmd.length]
-      dispatch_assert(cmd, expected)
+      set_zone_value('V', zone, volume)
+    end
+
+    def set_zone_mute(zone, state)
+      set_zone_value('M', zone, state ? 1 : 0)
     end
 
     def get_channel_volume(channel = nil)
@@ -79,12 +111,16 @@ module Sonamp
       dispatch_assert(cmd, expected)
     end
 
+    def set_channel_mute(channel, state)
+      set_channel_value('MC', channel, state ? 1 : 0)
+    end
+
     def get_zone_mute(zone = nil)
       get_zone_state('M', zone)
     end
 
     def get_channel_mute(channel = nil)
-      get_channel_value('MC', channel)
+      get_channel_state('MC', channel)
     end
 
     def get_channel_front_panel_level(channel = nil)
@@ -127,23 +163,23 @@ module Sonamp
       # Reusing the opened device file makes :VTIG? fail even with a delay
       # in front.
       #open_device do
+        {
+          firmware_version: get_firmware_version,
+          temperature: get_temperature,
+          zone_power: get_zone_power,
+          zone_fault: get_zone_fault,
+          zone_volume: get_zone_volume,
+          channel_volume: get_channel_volume,
+          zone_mute: get_zone_mute,
+          channel_mute: get_channel_mute,
+          bbe: get_bbe,
+          bbe_high_boost: get_bbe_high_boost,
+          bbe_low_boost: get_bbe_low_boost,
+          auto_trigger_input: get_auto_trigger_input,
+          voltage_trigger_input: get_voltage_trigger_input,
+          channel_front_panel_level: get_channel_front_panel_level,
+        }
       #end
-      {
-        firmware_version: get_firmware_version,
-        temperature: get_temperature,
-        zone_power: get_zone_power,
-        zone_fault: get_zone_fault,
-        zone_volume: get_zone_volume,
-        channel_volume: get_channel_volume,
-        zone_mute: get_zone_mute,
-        channel_mute: get_channel_mute,
-        bbe: get_bbe,
-        bbe_high_boost: get_bbe_high_boost,
-        bbe_low_boost: get_bbe_low_boost,
-        auto_trigger_input: get_auto_trigger_input,
-        voltage_trigger_input: get_voltage_trigger_input,
-        channel_front_panel_level: get_channel_front_panel_level,
-      }
     end
 
     private
@@ -152,7 +188,7 @@ module Sonamp
       if @f
         yield
       else
-        File.open(device, 'r+') do |f|
+        File.open(device, 'r+b') do |f|
           @f = f
           yield.tap do
             @f = nil
@@ -163,7 +199,9 @@ module Sonamp
 
     def dispatch(cmd, resp_lines_count = 1)
       open_device do
-        @f << "#{cmd}\x0d"
+        with_timeout do
+          @f.syswrite("#{cmd}\x0d")
+        end
         resp = 1.upto(resp_lines_count).map do
           read_line(@f, cmd)
         end
@@ -184,7 +222,7 @@ module Sonamp
 
     def extract_suffix(resp, expected_prefix)
       unless resp[0..expected_prefix.length-1] == expected_prefix
-        raise "Unexpected response: expected #{expected_prefix}..., actual #{resp}"
+        raise UnexpectedResponse, "Unexpected response: expected #{expected_prefix}..., actual #{resp}"
       end
       resp[expected_prefix.length..]
     end
@@ -198,44 +236,89 @@ module Sonamp
       dispatch_extract_suffix(":#{cmd}?", cmd)
     end
 
+    def with_timeout(&block)
+      Timeout.timeout(RS232_TIMEOUT, CommunicationTimeout, &block)
+    end
+
     def read_line(f, cmd)
-      f.readline.strip.tap do |resp|
-        if resp == 'ERR'
-          raise InvalidCommand, "Invalid command: #{cmd}"
-        elsif resp == 'N/A'
-          raise NotApplicable, "Command was recognized but could not be executed - is serial control enabled on the amplifier?"
+      with_timeout do
+        f.readline.strip.tap do |resp|
+          if resp == 'ERR'
+            raise InvalidCommand, "Invalid command: #{cmd}"
+          elsif resp == 'N/A'
+            raise NotApplicable, "Command was recognized but could not be executed - is serial control enabled on the amplifier?"
+          end
         end
       end
     end
 
-    def get_zone_state(cmd_prefix, zone)
+    def set_zone_value(cmd_prefix, zone, value)
+      if zone < 1 || zone > 4
+        raise ArgumentError, "Zone must be between 1 and 4: #{zone}"
+      end
+      cmd = ":#{cmd_prefix}#{zone}#{value}"
+      expected = cmd[1...cmd.length]
+      dispatch_assert(cmd, expected)
+    end
+
+    def get_zone_value(cmd_prefix, zone, boolize: false)
       if zone
         if zone < 1 || zone > 4
           raise ArgumentError, "Zone must be between 1 and 4: #{zone}"
         end
         resp = dispatch(":#{cmd_prefix}#{zone}?")
-        resp[cmd_prefix.length + 1] == '1' ? true : false
+        typecast_value(resp[cmd_prefix.length + 1..], boolize)
       else
-        dispatch(":#{cmd_prefix}G?", 4).map do |resp|
-          resp[cmd_prefix.length + 1] == '1' ? true : false
+        index = 1
+        hashize_query_result(dispatch(":#{cmd_prefix}G?", 4), cmd_prefix, boolize)
+      end
+    end
+
+    def hashize_query_result(resp_lines, cmd_prefix, boolize)
+      index = 1
+      Hash[resp_lines.map do |resp|
+        value = typecast_value(extract_suffix(resp, "#{cmd_prefix}#{index}"), boolize)
+        [index, value].tap do
+          index += 1
         end
+      end]
+    end
+
+    def get_zone_state(cmd_prefix, zone)
+      get_zone_value(cmd_prefix, zone, boolize: true)
+    end
+
+    def set_channel_value(cmd_prefix, channel, value)
+      if channel < 1 || channel > 8
+        raise ArgumentError, "Channel must be between 1 and 8: #{channel}"
       end
+      cmd = ":#{cmd_prefix}#{channel}#{value}"
+      expected = cmd[1...cmd.length]
+      dispatch_assert(cmd, expected)
     end
 
-    def get_channel_value(cmd_prefix, channel)
+    def get_channel_value(cmd_prefix, channel, boolize: false)
       if channel
         if channel < 1 || channel > 8
           raise ArgumentError, "Channel must be between 1 and 8: #{channel}"
         end
-        Integer(dispatch_extract_suffix(":#{cmd_prefix}#{channel}?", "#{cmd_prefix}#{channel}"))
+        typecast_value(dispatch_extract_suffix(":#{cmd_prefix}#{channel}?", "#{cmd_prefix}#{channel}"), boolize)
       else
         index = 1
-        dispatch(":#{cmd_prefix}G?", 8).map do |resp|
-          Integer(extract_suffix(resp, "#{cmd_prefix}#{index}")).tap do
-            index += 1
-          end
-        end
+        hashize_query_result(dispatch(":#{cmd_prefix}G?", 8), cmd_prefix, boolize)
+      end
+    end
+
+    def get_channel_state(cmd_prefix, channel)
+      get_channel_value(cmd_prefix, channel, boolize: true)
+    end
+
+    def typecast_value(value, boolize)
+      value = Integer(value)
+      if boolize
+        value = value == 1
       end
+      value
     end
   end
 end

data/lib/sonamp/cmd.rb

@@ -24,6 +24,7 @@ module Sonamp
     end
 
     attr_reader :args
+    attr_reader :logger
 
     def run
       cmd = args.shift
@@ -32,6 +33,15 @@ module Sonamp
       end
 
       case cmd
+      when 'detect'
+        device = Sonamp.detect_device(*args, logger: logger)
+        if device
+          puts device
+          exit 0
+        else
+          STDERR.puts("Sonamp amplifier not found")
+          exit 3
+        end
       when 'off'
         client.set_zone_power(1, false)
         client.set_zone_power(2, false)
@@ -49,6 +59,14 @@ module Sonamp
         channel = args.shift.to_i
         volume = args.shift.to_i
         client.set_channel_volume(channel, volume)
+      when 'zmute'
+        zone = args.shift.to_i
+        mute = ARGV.shift.to_i
+        client.set_zone_mute(zone, mute)
+      when 'cmute'
+        channel = args.shift.to_i
+        mute = args.shift.to_i
+        client.set_channel_mute(channel, mute)
       when 'status'
         pp client.status
       else

data/lib/sonamp/utils.rb

@@ -5,7 +5,7 @@ module Sonamp
       case value&.downcase
       when '1', 'on', 'yes', 'true'
         true
-      when '0', 'off', 'no', 'value'
+      when '0', 'off', 'no', 'false'
         false
       else
         raise ArgumentError, "Invalid on/off value: #{value}"

data/lib/sonamp/version.rb

@@ -1,3 +1,3 @@
 module Sonamp
-  VERSION = '0.0.5'.freeze
+  VERSION = '0.0.8'.freeze
 end

data/sonamp.gemspec

@@ -2,15 +2,15 @@
 
 Gem::Specification.new do |spec|
   spec.name          = "sonamp"
-  spec.version       = '0.0.5'
+  spec.version       = '0.0.8'
   spec.authors       = ['Oleg Pudeyev']
   spec.email         = ['code@olegp.name']
   spec.summary       = %q{Sonance Sonamp Amplifier Serial Control Interface}
-  spec.description   = %q{Library for controlling Sonance Sonamp 875D & 875D MkII amplifiers via the serial port}
+  spec.description   = %q{Library for controlling Sonance Sonamp 875D SE & 875D MkII amplifiers via the serial port}
   spec.homepage      = "https://github.com/p/sonamp-ruby"
   spec.license       = "MIT"
 
-  spec.files         = `git ls-files -z [A-Z]* bin lib *.gemspec`.split("\x0")
+  spec.files         = `git ls-files -z`.split("\x0").reject { |path| path.start_with?('docs/') }
   spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]

metadata

@@ -1,17 +1,17 @@
 --- !ruby/object:Gem::Specification
 name: sonamp
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.8
 platform: ruby
 authors:
 - Oleg Pudeyev
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-10-27 00:00:00.000000000 Z
+date: 2022-11-16 00:00:00.000000000 Z
 dependencies: []
-description: Library for controlling Sonance Sonamp 875D & 875D MkII amplifiers via
-  the serial port
+description: Library for controlling Sonance Sonamp 875D SE & 875D MkII amplifiers
+  via the serial port
 email:
 - code@olegp.name
 executables:
@@ -20,6 +20,7 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
+- ".gitignore"
 - LICENSE
 - bin/sonamp
 - bin/sonamp-web