procon_bypass_man 0.3.6 → 0.3.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2f19608c134a980243f4e45da1a1685367dd4bffc114482682b5d8d6bef03a90
-  data.tar.gz: c0ce801d735e63e5d6a385854ad83abb08fe044342bda63b5d70d2d50b3589f4
+  metadata.gz: 86e705d2907d51077191585d2cd16fb198a8a55226613d25f44b15b5e58f8806
+  data.tar.gz: a13290f301295913bbc30a06a1e06689dc5aae328cadd39ed1e91982ff7586d2
 SHA512:
-  metadata.gz: 11e84f9b7fa368adc1cbbb8593aca61d2a1a362453a13c5b226912d490c82933d476072f11ecb36226657107aa68b08be734ac19ec8901b56a00deb980d218b9
-  data.tar.gz: 642a6813619dccbb9d28f634bb7bb866ec92a2ed34fac254336562ed462512b1e9eb6ec2672dae17727e9c6790651f4e481e24a9a919dbd87381e4a457c62796
+  metadata.gz: 30618560c54d7c7a730bf97d74b601fce287d6900ed3eaf3c54b1071349b8a7d8b9e346c9099f6dbfad25a568710c5aaa4ce09ad4b12805a2f6d2d3a8134b27f
+  data.tar.gz: 96c796ecbbfa544a0427537a50ea17a0b7f10eee360bc6b072872c7621627dccf5403e78bdc2b580e5957fc5758f9b58979ef1f7f341da1a58b849a693ae79f8

data/.circleci/config.yml

@@ -119,6 +119,7 @@ build_jobs: &build_jobs
             - "3.0.1"
             - "3.0.2"
             - "3.1.2"
+            - "3.2.2"
   - bundle_install:
       matrix:
         parameters:
@@ -127,6 +128,7 @@ build_jobs: &build_jobs
             - "3.0.1"
             - "3.0.2"
             - "3.1.2"
+            - "3.2.2"
   - lint:
       matrix:
         parameters:
@@ -135,6 +137,7 @@ build_jobs: &build_jobs
             - "3.0.1"
             - "3.0.2"
             - "3.1.2"
+            - "3.2.2"
       requires:
         - bundle_install
   - type_check:
@@ -154,6 +157,7 @@ build_jobs: &build_jobs
             - "3.0.1"
             - "3.0.2"
             - "3.1.2"
+            - "3.2.2"
       requires:
         - bundle_install
 workflows:

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## [0.3.7] 2023-04-06
+* PBM-Cloudと連携しているかをbootメッセージに表示ようになりました
+* シリアルポート経由でPBMに入力ができるようになりました
+
 ## [0.3.6] 2023-03-12
 - プロコンの入力状態をpbm-cloudサーバに送信できるようになりました
 

data/Gemfile

@@ -13,6 +13,7 @@ gem "rubocop", require: false
 gem "sinatra", require: false
 gem "webrick", require: false
 gem "stackprof", require: false
+gem "serialport" # シリアル通信をする時に必要。通常はいらない
 
 if Gem::Version.new(RUBY_VERSION) > Gem::Version.new("2.6.0")
   gem 'typeprof', require: false

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    procon_bypass_man (0.3.6)
+    procon_bypass_man (0.3.7)
       action_cable_client
       blue_green_process (= 0.1.4.2)
       pbmenv (>= 0.1.11)
@@ -80,6 +80,7 @@ GEM
       parser (>= 3.0.1.1)
     ruby-progressbar (1.11.0)
     ruby2_keywords (0.0.5)
+    serialport (1.3.2)
     set (1.0.3)
     sinatra (2.2.0)
       mustermann (~> 1.0)
@@ -134,6 +135,7 @@ DEPENDENCIES
   rbs
   rspec
   rubocop
+  serialport
   sinatra
   stackprof
   steep

data/bin/console

@@ -14,6 +14,9 @@ require "pry"
 
 require "irb"
 
+# TODO: replのときチェックが入らないので、書く詳細な実装に触れないようにする。PBM_ENVを参照するなどをする
+ProconBypassMan::ExternalInput.prepare_channels
+
 ProconBypassMan.config.logger = Logger.new($stdout)
 
 IRB.start(__FILE__)

data/bin/validate_external_input

@@ -0,0 +1,19 @@
+#!/usr/bin/env ruby
+
+# Usage:
+#   NOTE: JSON形式の文字列がどういう形で読み込まれるかを確認できます
+#   $ echo '{"hex":"hogehoge", "buttons": ["a","b", "zr"]}' | bin/validate_external_input
+#   => 読み取った値: {:hex=>"hogehoge", :buttons=>["a", "b", "zr"]}
+
+require "bundler/setup"
+require "json"
+require "procon_bypass_man"
+require "procon_bypass_man/external_input"
+
+begin
+  json_str = ARGF.read
+  external_data = ProconBypassMan::ExternalInput::ExternalData.parse!(json_str)
+  puts("読み取った値: #{{ hex: external_data.hex, buttons: external_data.buttons }}")
+rescue JSON::ParserError => e
+  puts "failed to parse JSON: #{e.message}"
+end

data/docs/getting_started.md

@@ -18,6 +18,7 @@
 * [設定ファイルの書き方がわからない、エラーが起きるとき](#設定ファイルの書き方がわからない、エラーが起きるとき)
 * [procon_bypass_manのアップグレード方法](#procon_bypass_manのアップグレード方法)
 * [procon_bypass_man_cloudについて](#procon_bypass_man_cloudについて)
+* [シリアルポート・GPIOから読み取る](#シリアルポートから読み取る)
 * [最適化について](#最適化について)
 
 ## はじめに
@@ -251,6 +252,10 @@ procon_bypass_man_cloudとの接続が完了後、Raspberry Piを起動時にpro
   
 セットアップ方法は https://pbm-cloud.jiikko.com/faq に書いています。
 
+## シリアルポートから読み取る
+* [ラズベリーパイのシリアルポート(GPIO)へ書き込んでPBM経由してSwitchへ入力をする方法](/docs/setting/integration_external_input_serial_port.md)
+* [ラズベリーパイのシリアルポート(GPIO)に書き込むフォーマットについて](/docs/setting/integration_external_input_serial_port_format.md)
+
 ## 最適化について
 本稿では、Rubyの最適化について書きます。上級者向けです。適用しなくても普通に動きますが、逆に適用したことで何らかのケースで遅くなる場合があるかもしれません。
 

data/docs/setting/integration_external_input_serial_port.md

@@ -0,0 +1,56 @@
+# ラズベリーパイのシリアルポート(GPIO)へ書き込んでPBM経由してSwitchへ入力をする方法
+* procon_bypass_man: 0.3.7以上が必要です
+* GPIOからSwitchへ入力できます。本テキストでは設定方法を記載します
+
+## 1. シリアルポートへ書き込みができるようにラズベリーパイのセットアップする
+https://toki-blog.com/pi-serial/ の「汎用シリアルとして使う」 を実施してください
+
+## 2. GPIOへケーブルを挿す
+対応しているケーブルをGPIOとPCに接続してください。
+
+## 3. PBMのapp.rbを編集し、シリアルポートから読み出せるようにPBMのapp.rbを編集する
+`gemの追加`と`デバイスファイルを指す修正` の2つ必要です。
+
+* 1) シリアルポートから読み出すために追加でgemが必要です。 `app.rb` に `gem "serialport"` を追加してください。
+
+```diff
+  gemfile do
+    source 'https://rubygems.org'
+    git_source(:github) {|repo_name| "https://github.com/#{repo_name}" }
+    gem 'procon_bypass_man', '0.3.7'
++   gem "serialport"
+  end
+```
+
+* 2) 以下を参考にして`config.external_input_channels`の行を追加してください。
+
+```diff
+ProconBypassMan.configure do |config|
+  config.root = File.expand_path(__dir__)
+  config.logger = Logger.new("#{ProconBypassMan.root}/app.log", 1, 1024 * 1024 * 1)
+  config.logger.level = :debug
+
+  # バイパスするログを全部app.logに流すか
+  config.verbose_bypass_log = false
+
+  # webからProconBypassManを操作できるwebサービスと連携します
+  # 連携中はエラーログ、パフォーマンスに関するメトリクスを送信します
+  # config.api_servers = 'https://pbm-cloud.jiikko.com'
+
+  # エラーが起きたらerror.logに書き込みます
+  config.enable_critical_error_logging = true
+
+  # pbm-cloudで使う場合はnever_exitにtrueをセットしてください. trueがセットされている場合、不慮の事故が発生してもプロセスが終了しなくなります
+  config.never_exit_accidentally = true
+
+  # 接続に成功したらコントローラーのHOME LEDを光らせるか
+  config.enable_home_led_on_connect = true
+
++ config.external_input_channels = [
++   ProconBypassMan::ExternalInput::Channels::SerialPortChannel.new(device_path: '/dev/serial0', baud_rate: 9600),
++ ]
+end
+```
+
+以上で設定は完了です。PBMを起動し、連携ツールからGPIOへ書き込んでください。  
+書き込みフォーマットについては [ラズベリーパイのシリアルポート(GPIO)に書き込むフォーマットについて](/docs/setting/integration_external_input_serial_port_format.md) を参照してください。

data/docs/setting/integration_external_input_serial_port_format.md

@@ -0,0 +1,36 @@
+# ラズベリーパイのシリアルポート(GPIO)に書き込むフォーマットについて
+* procon_bypass_man: 0.3.7からGPIOを経由してSwitchへ入力できるようになりました。本テキストではPBMが読み取れるフォーマットについて記載します
+
+## フォーマット
+JSON形式, plain textに対応しています
+
+* JSON形式
+    * 受付可能なカラム
+      * hex
+        * 16進数でエンコードした`INPUT 0x30` を書いてください。スティック操作を除くボタン部分のみが読み込まれます
+        * https://github.com/dekuNukem/Nintendo_Switch_Reverse_Engineering/blob/ac8093c84194b3232acb675ac1accce9bcb456a3/bluetooth_hid_notes.md#input-0x30
+        * ex: '30f28100800078c77448287509550274ff131029001b0022005a0271ff191028001e00210064027cff1410280020002100000000000000000000000000000000'
+      * buttons
+        * 「入力したいボタン」「入力したくないボタン」を配列で渡してください
+        * 「入力したいボタン」
+             * `"y", "x", "b", "a", "sl", "sr", "r", "zr", "minus", "plus", "thumbr", "thumbl", "home", "cap", "down", "up", "right", "left", "l", "zl"`
+        * 「入力したくないボタン」
+             * `"uny", "unx", "unb", "una", "unsl", "unsr", "unr", "unzr", "unminus", "unplus", "unthumbr", "unthumbl", "unhome", "uncap", "undown", "unup", "unright", "unleft", "unl", "unzl"`
+        * ex: `['a', 'b']`
+    * 仕様
+       * JSONにhex, buttonsの両方が含まれている場合、hexを優先し、buttonsは無視します
+    * JSONの例
+      * `'{"hex":"30f28100800078c77448287509550274ff131029001b0022005a0271ff191028001e00210064027cff1410280020002100000000000000000000000000000000", "buttons": ["a"] }'`
+* plain text
+  * コロンで囲ったボタンを1つの入力として解釈します
+  * `:a::b::uny:` を入力した場合には、a, b, uny を読み込みます
+
+## 正しいJSONかを検証する
+本リポジトリにJSONを検証する `bin/validate_external_input` を同梱しています。実行例を以下に示します。
+
+```shell
+$ echo '{"hex":"30f28100800078c77448287509550274ff131029001b0022005a0271ff191028001e00210064027cff1410280020002100000000000000000000000000000000", "buttons": ["a"] }' | bin/validate_external_input
+読み取った値: {:hex=>"30f28100800078c77448287509550274ff131029001b0022005a0271ff191028001e00210064027cff1410280020002100000000000000000000000000000000", :buttons=>[:a]}
+```
+
+以上。

data/lib/procon_bypass_man/background/jobs/report_procon_performance_measurements_job.rb

@@ -21,6 +21,7 @@ class ProconBypassMan::ReportProconPerformanceMeasurementsJob < ProconBypassMan:
       time_taken_p50: metric.time_taken_p50,
       time_taken_p95: metric.time_taken_p95,
       time_taken_p99: metric.time_taken_p99,
+      external_input_time_max: metric.external_input_time_max,
       read_error_count: metric.read_error_count,
       write_error_count: metric.write_error_count,
       gc_count: metric.gc_count,

data/lib/procon_bypass_man/bypass/procon_to_switch.rb

@@ -50,14 +50,29 @@ class ProconBypassMan::Bypass::ProconToSwitch
         # 後続処理で入力値を取得できるように詰めておく
         ProconBypassMan::ProconDisplay::Status.instance.current = bypass_value.binary.to_procon_reader.to_hash
 
+        # NOTE: 外部からの入力を受け取る
+        external_input_data = nil
+        measurement.record_external_input_time do
+          # TODO: シリアルぽーとから読み取ると252.chrみたいなゴミデータを受け取ってEncoding::UndefinedConversionErrorが発生する可能性がある. 発生したら上限までretryした方がいいかも
+          if(data = ProconBypassMan::ExternalInput.read)
+            begin
+              external_input_data = ProconBypassMan::ExternalInput::ExternalData.parse!(data)
+              ProconBypassMan.logger.debug { "[ExternalInput] 読み取った値: { hex: #{external_input_data.hex}, raw_data: '#{external_input_data.raw_data}', buttons: #{external_input_data.buttons} }" }
+            rescue ProconBypassMan::ExternalInput::ParseError => e
+              ProconBypassMan.logger.error "[ExternalInput][#{e}] #{data.force_encoding('UTF-8').scrub}, #{data.force_encoding('ASCII-8BIT').codepoints} をparseできませんでした"
+            end
+          end
+        end
+
         result = measurement.record_write_time do
           begin
             ProconBypassMan::Retryable.retryable(tries: 5, on_no_retry: [Errno::EIO, Errno::ENODEV, Errno::EPROTO, IOError, Errno::ESHUTDOWN, Errno::ETIMEDOUT]) do
               begin
                 # 終了処理を希望されているのでブロックを無視してメソッドを抜けてOK
                 return(false) if will_terminate? # rubocop:disable Lint/NoReturnInBeginEndBlocks
+
                 binary = ::ProconBypassMan::Procon::Rumbler.monitor do
-                  ProconBypassMan::Processor.new(bypass_value.binary).process
+                  ProconBypassMan::Processor.new(bypass_value.binary).process(external_input_data: external_input_data)
                 end
                 self.gadget.write_nonblock(binary)
 

data/lib/procon_bypass_man/commands/print_boot_message_command.rb

@@ -34,6 +34,7 @@ class ProconBypassMan::PrintBootMessageCommand
       ProconBypassMan::VERSION: #{@table[:pbm_version]}
       RUBY_VERSION: #{@table[:ruby_version]}
       Pbmenv::VERSION: #{@table[:pbmenv_version]}
+      PBM-Cloud Integration: #{ProconBypassMan::WebConnectivityChecker.new(ProconBypassMan.config.api_server)}
       pid: #{@table[:pid]}
       root: #{@table[:root_path]}
       pid_path: #{@table[:pid_path]}
@@ -56,6 +57,7 @@ class ProconBypassMan::PrintBootMessageCommand
   def self.execute
     message = BootMessage.new
     ProconBypassMan::ReportBootJob.perform_async(message.to_hash)
+    ProconBypassMan.logger.info message.to_s
     puts message.to_s
   end
 end

data/lib/procon_bypass_man/configuration.rb

@@ -54,6 +54,7 @@ class ProconBypassMan::Configuration
 
   attr_accessor :enable_critical_error_logging
   attr_writer :verbose_bypass_log, :raw_setting, :never_exit_accidentally, :enable_home_led_on_connect
+  attr_writer :external_input_channels
   # 削除予定
   attr_writer :enable_reporting_pressed_buttons
 
@@ -193,4 +194,9 @@ class ProconBypassMan::Configuration
       true
     end
   end
+
+  # @return [Array<ProconBypassMan::ExternalInput::Channel::TCPIP, ProconBypassMan::ExternalInput::Channel::SerialPort>]
+  def external_input_channels
+    @external_input_channels || []
+  end
 end

data/lib/procon_bypass_man/external_input/channels/base.rb

@@ -0,0 +1,12 @@
+module ProconBypassMan
+  module ExternalInput
+    module Channels
+      class Base
+        # @return [String, NilClass]
+        def read
+          raise NotImplementedError
+        end
+      end
+    end
+  end
+end

data/lib/procon_bypass_man/external_input/channels/serial_port_channel.rb

@@ -0,0 +1,48 @@
+module ProconBypassMan
+  module ExternalInput
+    module Channels
+      class SerialPortChannel < Base
+        attr_reader :serial_port
+
+        # @param [String] device_path
+        # @param [Integer] baud_rate
+        def initialize(device_path: , baud_rate: 9600)
+          require 'serialport'
+
+          super()
+          # data_bitsあたりは必要があれば設定ができるようにしたいがよくわからないのでとりあえずnilを入れる
+          data_bits = 8
+          stop_bits = 1
+          parity = SerialPort::NONE
+          @serial_port= SerialPort.new(device_path, baud_rate, data_bits, stop_bits, parity)
+        end
+
+        # @return [String, NilClass]
+        # NOTE: この実装では、::IO::EAGAINWaitReadableを実質的な終端として扱っているので、高速に書き込みがされると取りこぼす可能性がある.
+        # NOTE: read_nonblockでバッファから読み出すとき、バッファが空になるまでは::IO::EAGAINWaitReadableが起きない前提で実装している.
+        # NOTE: 取りこぼししないよう精度を上げるには、終端文字が来るまで1文字ずつ読む必要があるがパフォーマンスが犠牲になってしまう. この対策をするには、bypass処理の開始で非同期に1文字ずつ読み込むことをすると多少マシになるはず
+        def read
+          buffer = ''
+          loop do
+            begin
+              buffer += @serial_port.read_nonblock(1024) || ''
+            rescue ::IO::EAGAINWaitReadable
+              break
+            end
+          end
+
+          return nil if buffer.empty?
+
+          # NOTE: 高速に書き込まれた場合、複数のチャンクを含む可能性があるので、最初だけを切り取る
+          chunks = buffer.split("\n")
+          if(chunks.size > 1)
+            ProconBypassMan::SendErrorCommand.execute(
+              error: "[ExternalInput] シリアルポートから読み込んだchunkが複数あります. 高い書き込み頻度に耐えられていないので実装を見直してください。 (chunks.size: #{chunks.size})"
+            )
+          end
+          chunks.first
+        end
+      end
+    end
+  end
+end

data/lib/procon_bypass_man/external_input/channels/tcpip.rb

@@ -0,0 +1,16 @@
+# TODO: 実装する
+# module ProconBypassMan
+#   module ExternalInput
+#     module Channels
+#       class TCPIP < Base
+#         def initialize(port: )
+#           super()
+#           # TODO: ここでTCPサーバを起動する(port)
+#         end
+#
+#         def read
+#         end
+#       end
+#     end
+#   end
+# end

data/lib/procon_bypass_man/external_input/channels.rb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+module ProconBypassMan
+  module ExternalInput
+    module Channels
+    end
+  end
+end
+
+require "procon_bypass_man/external_input/channels/base"
+require "procon_bypass_man/external_input/channels/serial_port_channel"
+require "procon_bypass_man/external_input/channels/tcpip"

data/lib/procon_bypass_man/external_input/external_data.rb

@@ -0,0 +1,79 @@
+module ProconBypassMan
+  module ExternalInput
+    class ExternalData
+      UNPRESS_BUTTONS = Set.new(ProconBypassMan::Procon::ButtonCollection.available.map { |x| "un#{x}".to_sym })
+
+      # @return [String, NilClass] 16進数表現のデータ
+      attr_reader :hex
+
+      # @return [String, NilClass] ログに表示する用
+      attr_reader :raw_data
+
+      # @raise [ParseError]
+      # @return [ExternalData] JSON か カンマ区切りのbuttons
+      def self.parse!(raw_data)
+        raise ParseError unless raw_data.ascii_only?
+
+        if is_json(raw_data)
+          begin
+            json = JSON.parse(raw_data)
+            return new(hex: json['hex'], buttons: json['buttons'])
+          rescue JSON::ParserError
+            raise ParseError
+          end
+        end
+
+        return new(hex: nil, buttons: raw_data.scan(/:\w+:/).map { |x| x.gsub(':', '') }, raw_data: raw_data)
+      end
+
+      # @param [String] raw_data
+      # @return [Boolean]
+      def self.is_json(raw_data)
+        raw_data.start_with?('{')
+      end
+
+      # @param [String, NilClass] hex
+      # @param [Array<String>, NilClass] buttons
+      # @param [String, NilClass] raw_data
+      def initialize(hex: , buttons: , raw_data: nil)
+        @hex = hex
+        @buttons = buttons || []
+        @raw_data = raw_data
+      end
+
+      # @return [String, NilClass]
+      def to_binary
+        return nil if @hex.nil?
+        [@hex].pack('H*')
+      end
+
+      # @return [Array<Symbol>]
+      def press_buttons
+        buttons.select do |button|
+          ProconBypassMan::Procon::ButtonCollection::BUTTONS_MAP[button]
+        end
+      end
+
+      # @return [Array<Symbol>]
+      def unpress_buttons
+        buttons.select { |button|
+          UNPRESS_BUTTONS.include?(button)
+        }.map { |b| to_button(b.to_s).to_sym }
+      end
+
+      # NOTE: ログに表示する用
+      # @return [Array<Symbol>]
+      def buttons
+        @buttons.map(&:to_sym)
+      end
+
+      private
+
+      # @return [String]
+      # NOTE: un#{button} って名前をbuttonに変換する
+      def to_button(button)
+        button.sub(/^un/, '')
+      end
+    end
+  end
+end

data/lib/procon_bypass_man/external_input.rb

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+module ProconBypassMan
+  module ExternalInput
+    class ParseError < StandardError; end
+
+    @@channels = nil
+
+    # @return [Array<ProconBypassMan::ExternalInput::Channels::Base>]
+    def self.channels
+      @@channels
+    end
+
+    # @return [void]
+    def self.prepare_channels
+      @@channels = ProconBypassMan.config.external_input_channels
+    end
+
+    # @return [NilClass, String]
+    # NOTE: 外部入力からのreadがボトルネックになるなら、Concurrent::Futureを使ってプロコンからの読み出しと並列化することを検討する
+    def self.read
+      raise '外部入力が未初期化です' if @@channels.nil?
+
+      value = nil
+      @@channels.each do |channel|
+        value = channel.read
+        break if value
+      end
+      value
+    end
+  end
+end
+
+require "procon_bypass_man/external_input/external_data"
+require "procon_bypass_man/external_input/channels.rb"

data/lib/procon_bypass_man/processor.rb

@@ -5,12 +5,13 @@ class ProconBypassMan::Processor
     @binary = binary
   end
 
+  # @param [ProconBypassMan::ExternalInput::ExternalData, NilClass] external_input_data
   # @return [String] 加工後の入力データ
-  def process
+  def process(external_input_data: nil)
     return @binary.raw unless @binary.user_operation_data?
 
     procon = ProconBypassMan::Procon.new(@binary.raw)
     procon.apply!
-    procon.to_binary
+    procon.to_binary(external_input_data: external_input_data)
   end
 end

data/lib/procon_bypass_man/procon/button_collection.rb

@@ -37,4 +37,9 @@ class ProconBypassMan::Procon::ButtonCollection
   BUTTONS = ProconBypassMan::Procon::ButtonCollection::BUTTONS_MAP.keys.freeze
 
   LEFT_ANALOG_STICK = { byte_position: 6..8 }
+
+  # @return [Array<Symbol>]
+  def self.available
+    BUTTONS
+  end
 end

data/lib/procon_bypass_man/procon/performance_measurement/measurements_summarizer.rb

@@ -9,6 +9,7 @@ class ProconBypassMan::Procon::PerformanceMeasurement::MeasurementsSummarizer
                                        :time_taken_p95,
                                        :time_taken_p99,
                                        :time_taken_max,
+                                       :external_input_time_max,
                                        :read_error_count,
                                        :write_error_count,
                                        :gc_count,
@@ -26,6 +27,7 @@ class ProconBypassMan::Procon::PerformanceMeasurement::MeasurementsSummarizer
     write_time_max = 0
     read_time_max = 0
     time_taken_max = 0
+    external_input_time_max = 0
     interval_from_previous_succeed_max = 0
     @spans.each do |span|
       # NOTE @spans.map(&:write_time).sort.last と同じことだけど、処理コストを軽くするためにループを共通化する
@@ -33,6 +35,7 @@ class ProconBypassMan::Procon::PerformanceMeasurement::MeasurementsSummarizer
       read_time_max = span.read_time if write_time_max < span.read_time
       time_taken_max = span.time_taken if span.succeed && time_taken_max < span.time_taken
       interval_from_previous_succeed_max = span.interval_from_previous_succeed if span.succeed && interval_from_previous_succeed_max < span.interval_from_previous_succeed
+      external_input_time_max = span.external_input_time if span.succeed && external_input_time_max < span.external_input_time
     end
 
     # NOTE 今はGCを無効にしており、集計するまでもないのでコメントアウトにする. 今後GCを有効にしたバイパスをするかもしれないので残しておく
@@ -79,6 +82,7 @@ class ProconBypassMan::Procon::PerformanceMeasurement::MeasurementsSummarizer
                           time_taken_p95,
                           time_taken_p99,
                           time_taken_max,
+                          external_input_time_max,
                           total_read_error_count,
                           total_write_error_count,
                           gc_count,

data/lib/procon_bypass_man/procon/performance_measurement.rb

@@ -12,7 +12,7 @@ require 'procon_bypass_man/procon/performance_measurement/last_bypass_at'
 module ProconBypassMan::Procon::PerformanceMeasurement
   class PerformanceSpan
     attr_accessor :time_taken, :succeed, :interval_from_previous_succeed, :gc_count, :gc_time
-    attr_reader :write_error_count, :read_error_count, :write_time, :read_time
+    attr_reader :write_error_count, :read_error_count, :write_time, :read_time, :external_input_time
 
     def initialize
       @write_error_count = 0
@@ -40,9 +40,15 @@ module ProconBypassMan::Procon::PerformanceMeasurement
       return result
     end
 
+    # @return [void]
     def record_read_time(&block)
       @read_time = Benchmark.realtime { block.call }
     end
+
+    # @return [void]
+    def record_external_input_time(&block)
+      @external_input_time = Benchmark.realtime { block.call }
+    end
   end
 
   # 全部送ると負荷になるので適当にまびく

data/lib/procon_bypass_man/procon.rb

@@ -128,7 +128,7 @@ class ProconBypassMan::Procon
     end
 
     # remote macro or pbm action
-    if task = ProconBypassMan::RemoteAction::TaskQueueInProcess.non_blocking_shift
+    if(task = ProconBypassMan::RemoteAction::TaskQueueInProcess.non_blocking_shift)
       case task.type
       when ProconBypassMan::RemoteAction::Task::TYPE_MACRO
         no_op_step = :wait_for_0_3 # マクロの最後に固まって最後の入力をし続けるので、無の状態を最後に注入する
@@ -187,12 +187,27 @@ class ProconBypassMan::Procon
     status
   end
 
+  # @param [ProconBypassMan::ExternalInput::ExternalData, NilClass] external_input_data
   # @return [String]
-  def to_binary
+  def to_binary(external_input_data: nil)
     if ongoing_mode.name != :manual
       return user_operation.binary.raw
     end
 
+    if external_input_data
+      if(external_input_data_raw_binary = external_input_data.to_binary)
+        self.user_operation.merge(external_input_data_raw_binary)
+      else
+        external_input_data.press_buttons.each do |button|
+          self.user_operation.press_button(button)
+        end
+        external_input_data.unpress_buttons.each do |button|
+          self.user_operation.unpress_button(button)
+        end
+      end
+      return self.user_operation.binary.raw
+    end
+
     if ongoing_macro.ongoing? && (step = ongoing_macro.next_step)
       BlueGreenProcess::SharedVariable.extend_run_on_this_process = true
       ongoing_macro.force_neutral_buttons&.each do |force_neutral_button|

data/lib/procon_bypass_man/procon_display/status.rb

@@ -1,5 +1,3 @@
-require 'singleton'
-
 class ProconBypassMan::ProconDisplay::Status
   include Singleton
 

data/lib/procon_bypass_man/support/retryable.rb

@@ -1,14 +1,16 @@
 module ProconBypassMan
   class Retryable
-    def self.retryable(tries: , retried: 0, on_no_retry: [])
+    def self.retryable(tries: , retried: 0, on_no_retry: [], log_label: nil)
       return yield(retried)
     rescue *on_no_retry
       raise
-    rescue
+    rescue => e
       if tries <= retried
         raise
       else
         retried = retried + 1
+        ProconBypassMan.logger.debug "[Retryable]#{log_label && "[#{log_label}]"} #{e}が起きました。retryします。#{retried} / #{tries}"
+
         retry
       end
     end

data/lib/procon_bypass_man/support/web_connectivity_checker.rb

@@ -0,0 +1,39 @@
+class ProconBypassMan::WebConnectivityChecker
+  # @param [String, NilClass] url
+  def initialize(url)
+    @url = url
+  end
+
+  # @return [String]
+  def to_s
+    if @url.nil?
+      return "DISABLE"
+    end
+
+    if alive?
+      return "ENABLE (#{@url})"
+    else
+     return "UNREACHABLE (#{@url})"
+    end
+  end
+
+  private
+
+  # @return [Boolean]
+  def alive?
+    uri = URI.parse(@url)
+    response = nil
+
+    begin
+      Net::HTTP.start(uri.host, uri.port, use_ssl: uri.scheme == 'https') do |http|
+        request = Net::HTTP::Head.new(uri)
+        response = http.request(request)
+      end
+    rescue StandardError => e
+      ProconBypassMan.logger.error e
+      return false
+    end
+
+    response.is_a?(Net::HTTPSuccess) or response.is_a?(Net::HTTPMovedPermanently)
+  end
+end

data/lib/procon_bypass_man/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ProconBypassMan
-  VERSION = "0.3.6"
+  VERSION = "0.3.7"
 end

data/lib/procon_bypass_man.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require 'singleton'
 require "logger"
 require 'yaml'
 require "json"
@@ -40,6 +41,7 @@ require_relative "procon_bypass_man/support/cycle_sleep"
 require_relative "procon_bypass_man/support/can_over_process"
 require_relative "procon_bypass_man/support/retryable"
 require_relative "procon_bypass_man/support/renice_command"
+require_relative "procon_bypass_man/support/web_connectivity_checker"
 require_relative "procon_bypass_man/procon_display"
 require_relative "procon_bypass_man/background"
 require_relative "procon_bypass_man/commands"
@@ -61,7 +63,7 @@ require_relative "procon_bypass_man/worker"
 require_relative "procon_bypass_man/websocket/client"
 require_relative "procon_bypass_man/websocket/watchdog"
 require_relative "procon_bypass_man/websocket/forever"
-
+require_relative "procon_bypass_man/external_input"
 require_relative "procon_bypass_man/remote_action"
 
 STDOUT.sync = true
@@ -122,6 +124,8 @@ module ProconBypassMan
       return
     end
 
+    ProconBypassMan::ExternalInput.prepare_channels
+
     ready_pbm
     Runner.new(gadget: gadget, procon: procon).run # ここでblockingする
     terminate_pbm
@@ -130,6 +134,8 @@ module ProconBypassMan
   # 実行ファイル(app.rb)から呼び出している
   # @return [void]
   def self.configure(&block)
+    require_relative "procon_bypass_man/external_input"
+
     @@configuration = ProconBypassMan::Configuration.new
     @@configuration.instance_eval(&block)
     nil

data/project_template/app.rb

@@ -12,7 +12,11 @@ begin
   gemfile do
     source 'https://rubygems.org'
     git_source(:github) {|repo_name| "https://github.com/#{repo_name}" }
-    gem 'procon_bypass_man', '0.3.6'
+    gem 'procon_bypass_man', '0.3.7'
+    # uncomment if you want to use master branch
+    # gem 'procon_bypass_man', github: 'splaplapla/procon_bypass_man', branch: 'master'
+    # uncomment if you want to use serial communication feature
+    # gem "serialport"
   end
 rescue Bundler::Source::Git::GitCommandError => e
   retry_count_on_git_command_error = retry_count_on_git_command_error + 1
@@ -49,6 +53,11 @@ ProconBypassMan.configure do |config|
 
   # 接続に成功したらコントローラーのHOME LEDを光らせるか
   config.enable_home_led_on_connect = true
+
+  # シリアル通信やTCP/IP経由で入力するときに設定してください
+  # config.external_input_channels = [
+  #   ProconBypassMan::ExternalInput::Channels::SerialPortChannel.new(device_path: '/dev/serial0', baud_rate: 9600),
+  # ]
 end
 
 ProconBypassMan.run(setting_path: "/usr/share/pbm/current/setting.yml")

data/project_template/app.rb.erb

@@ -12,7 +12,11 @@ begin
   gemfile do
     source 'https://rubygems.org'
     git_source(:github) {|repo_name| "https://github.com/#{repo_name}" }
-    gem 'procon_bypass_man', '0.3.6'
+    gem 'procon_bypass_man', '0.3.7'
+    # uncomment if you want to use master branch
+    # gem 'procon_bypass_man', github: 'splaplapla/procon_bypass_man', branch: 'master'
+    # uncomment if you want to use serial communication feature
+    # gem "serialport"
   end
 rescue Bundler::Source::Git::GitCommandError => e
   retry_count_on_git_command_error = retry_count_on_git_command_error + 1
@@ -54,6 +58,12 @@ ProconBypassMan.configure do |config|
 
   # 接続に成功したらコントローラーのHOME LEDを光らせるか
   config.enable_home_led_on_connect = true
+
+  # シリアル通信やTCP/IP経由で入力するときに設定してください
+  # config.external_input_channels = [
+  #   ProconBypassMan::ExternalInput::Channels::SerialPortChannel.new(device_path: '/dev/serial0', baud_rate: 9600),
+  <%-# ProconBypassMan::ExternalInput::Channels::TCPIP.new(port: 9000), # TODO: まだ実装していない -%>
+  # ]
 end
 
 ProconBypassMan.run(setting_path: "/usr/share/pbm/current/setting.yml")

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: procon_bypass_man
 version: !ruby/object:Gem::Version
-  version: 0.3.6
+  version: 0.3.7
 platform: ruby
 authors:
 - jiikko
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2023-03-12 00:00:00.000000000 Z
+date: 2023-04-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pbmenv
@@ -95,9 +95,12 @@ files:
 - bin/generate_default_app
 - bin/generate_output_report
 - bin/setup
+- bin/validate_external_input
 - docs/getting_started.md
 - docs/how_to_connect_procon.md
 - docs/run_pbm_on_my_branch.md
+- docs/setting/integration_external_input_serial_port.md
+- docs/setting/integration_external_input_serial_port_format.md
 - docs/setting/left-analogstick-cap.md
 - docs/setting/splatoon2_macro_dasei_cancel.md
 - docs/setting/splatoon2_macro_forward_ikarole.md
@@ -176,6 +179,12 @@ files:
 - lib/procon_bypass_man/device_model.rb
 - lib/procon_bypass_man/device_status.rb
 - lib/procon_bypass_man/ephemeral_configuration.rb
+- lib/procon_bypass_man/external_input.rb
+- lib/procon_bypass_man/external_input/channels.rb
+- lib/procon_bypass_man/external_input/channels/base.rb
+- lib/procon_bypass_man/external_input/channels/serial_port_channel.rb
+- lib/procon_bypass_man/external_input/channels/tcpip.rb
+- lib/procon_bypass_man/external_input/external_data.rb
 - lib/procon_bypass_man/plugin/splatoon2/macro/charge_tansan_bomb.rb
 - lib/procon_bypass_man/plugin/splatoon2/macro/dasei_cancel.rb
 - lib/procon_bypass_man/plugin/splatoon2/macro/fast_return.rb
@@ -281,6 +290,7 @@ files:
 - lib/procon_bypass_man/support/update_remote_pbm_job_status_http_client.rb
 - lib/procon_bypass_man/support/uptime.rb
 - lib/procon_bypass_man/support/usb_device_controller.rb
+- lib/procon_bypass_man/support/web_connectivity_checker.rb
 - lib/procon_bypass_man/support/yaml_loader.rb
 - lib/procon_bypass_man/support/yaml_writer.rb
 - lib/procon_bypass_man/version.rb