procon_bypass_man 0.3.3.1 → 0.3.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 670f02fde81fd8b24214ec52a6863b08721c97959d5c5e52d6cc78efa4ab3749
-  data.tar.gz: 217a38476e7dd732fb692b89541daea156d9b389ee7677bf03f29126729f8ca2
+  metadata.gz: d4ffa6c5a0eca5298143505c133f6895f7bb33b6061a36e6123c27e19a0ffcbb
+  data.tar.gz: 74e77a823d913769a75b2dd698991bb1fed6a62a1da9a65928e00b86773e90df
 SHA512:
-  metadata.gz: 2f28ce822ca7ff9d3ece29e7aacad4b721c033c93792c6613b207d6bdfeaac7f0cbdf55247c7df121c783166562b429a24060f2103ef0efeaddcb9e615d54f22
-  data.tar.gz: f118e303fb62252263cdb16fde94d31134f68745bbabfa3519a2e3aa652603e2f9dfe6609bd3e3bf8a1e4cdfdb64e15bdbf205db4c12c549b0ac1a2182ced0d1
+  metadata.gz: 90cbb299a402bbab3fae219fc17b359155b48c67cf34761ad69c63ee5ba72ed7cf356d5829ee003d19558c7418f07444d7f35619d82bebc124975ab1c40cb85f
+  data.tar.gz: f03f39550078285114b0a7ac209938dee7b335c99689cba48ed4d4f4d62e659b0fb1fcea51bd800acfe61839cc92c235f5112e3adc5a3896e033f5dbfa224373

data/.circleci/config.yml

@@ -27,7 +27,8 @@ commands:
             - gem-cache-v1-<< parameters.ruby-version >>-{{ arch }}-{{ .Branch }}-{{ checksum "Gemfile.lock" }}
             - gem-cache-v1-<< parameters.ruby-version >>-{{ arch }}-{{ .Branch }}
             - gem-cache-v1
-      - run: gem i bundler && bundle install --path vendor/bundle --jobs 100 && bundle clean
+      # NOTE: ruby2.5だとbundler2.4系に対応していないのでバージョンを固定する
+      - run: gem i bundler -v 2.3.26 && bundle install --path vendor/bundle --jobs 100 && bundle clean
       - save_cache:
           key: gem-cache-v1-<< parameters.ruby-version >>-{{ arch }}-{{ .Branch }}-{{ checksum "Gemfile.lock" }}
           paths:

data/CHANGELOG.md

@@ -1,3 +1,14 @@
+## [0.3.5] - 2022-2-9
+* 設定ファイルに起動な必要なPBMのバージョンを表記できるようになりました
+  * https://github.com/splaplapla/procon_bypass_man/pull/238
+* レイヤー変更ボタン(prefix_keys_for_changing_layer)が空欄の時に設定ファイルをエラーにしないようになりました
+* PBM-CloudからPBMをアップグレードするときに、古いバージョンを削除するようにしました
+
+## [0.3.4] - 2022-12-26
+* 左スティックを1回転するマクロを追加しました
+    * 詳しい設定方法については docs/setting/splatoon3_rotation_left_stick.md を参照してください
+* プログラムの停止時・再読み込み時に時間がかかってた不具合を修正しました
+
 ## [0.3.3.1] - 2022-11-13
 * setting.ymlにmacroを使って「スプラ用マクロ、惰性キャンセル」の閾値を設定できない不具合を修正しました
 
@@ -14,7 +25,7 @@
 * レイヤーを変更しているときにコントローラーを振動できるようになりました
     * setting.yml で `enable(:rumble_on_layer_change)` と記述してください
 * イカロールをするマクロを追加しました
-    * 詳しい設定方法については docs/setting/splatoon2_macro_forward_ikarole.md を参照してください
+    * 詳しい設定方法については docs/setting/splatoon3_macro_forward_ikarole.md を参照してください
 * スプラ用マクロ、惰性キャンセルを削除しました
 
 ## [0.3.0] - 2022-9-9

data/Gemfile

@@ -12,6 +12,7 @@ gem "timecop"
 gem "rubocop", require: false
 gem "sinatra", require: false
 gem "webrick", require: false
+gem "stackprof", require: false
 
 if Gem::Version.new(RUBY_VERSION) > Gem::Version.new("2.6.0")
   gem 'typeprof', require: false

data/Gemfile.lock

@@ -1,10 +1,10 @@
 PATH
   remote: .
   specs:
-    procon_bypass_man (0.3.3.1)
+    procon_bypass_man (0.3.5)
       action_cable_client
-      blue_green_process
-      pbmenv (>= 0.1.9)
+      blue_green_process (= 0.1.4.2)
+      pbmenv (>= 0.1.11)
       sorted_set
 
 GEM
@@ -19,7 +19,7 @@ GEM
       tzinfo (~> 2.0)
       zeitwerk (~> 2.3)
     ast (2.4.2)
-    blue_green_process (0.1.3)
+    blue_green_process (0.1.4.2)
     coderay (1.1.3)
     concurrent-ruby (1.1.9)
     diff-lcs (1.4.4)
@@ -38,7 +38,7 @@ GEM
     parallel (1.21.0)
     parser (3.0.3.2)
       ast (~> 2.4.1)
-    pbmenv (0.1.10)
+    pbmenv (0.1.11)
     pry (0.14.1)
       coderay (~> 1.1)
       method_source (~> 1.0)
@@ -51,7 +51,7 @@ GEM
     rb-inotify (0.10.1)
       ffi (~> 1.0)
     rbs (1.7.1)
-    rbtree (0.4.5)
+    rbtree (0.4.6)
     regexp_parser (2.2.0)
     rexml (3.2.5)
     rspec (3.10.0)
@@ -89,6 +89,7 @@ GEM
     sorted_set (1.0.3)
       rbtree
       set (~> 1.0)
+    stackprof (0.2.22)
     steep (0.47.0)
       activesupport (>= 5.1)
       language_server-protocol (>= 3.15, < 4.0)
@@ -134,6 +135,7 @@ DEPENDENCIES
   rspec
   rubocop
   sinatra
+  stackprof
   steep
   timecop
   typeprof

data/README.md

@@ -38,7 +38,7 @@ https://user-images.githubusercontent.com/1664497/171327108-f12f56a5-fc36-48da-9
 * レイヤーを切り替える方法は？
     * 設定ファイルに記述している `prefix_keys_for_changing_layer`の後ろにあるキーを同時押しながら、十字キーのどれかを押すことで任意のレイヤーに切り替わります
 * このツールでできることは？
-    * キーリマップ, 連射, マクロ, 特定の同じ操作の繰り返し(mode)
+    * キーリマップ, 連射, マクロ
         * リマップは1つのキーを別のキーに割り当てます
     * 連射中には特定のキーの入力を無視したり、複数のキーをトリガーに連射することができます
 * どうしてsudoが必要なの？
@@ -52,7 +52,7 @@ https://user-images.githubusercontent.com/1664497/171327108-f12f56a5-fc36-48da-9
 * 日を跨ぐ24時ちょうどになった瞬間はLinuxのcronが起動などがするようで、この時間は数秒間バイパスが激しく遅延します
   * ログファイルのローテションが少なくとも走るはずなので、不要なデーモンを停止するなどで影響を小さくすることはできると思いますが、完全に抑制することは難しいと思います
 * コントローラーから読み取ってSwitchに書き込む時間は、少なくとも0.02秒はかかります
-  * 動かしているRaspberry Piの負荷に依存するこの限りではありません
+  * 遅延は動かしているRaspberry Piの負荷に依存します
 
 <!--
 ## TODO
@@ -80,13 +80,6 @@ sudo kill -USR2 `cat ./pbm_pid`
 * githubのreleaseを作成する
 -->
 
-## Raspberry Pi4のセットアップを代行します
-* Raspberry Pi4を開発者である私の家に配送してくれれば、セットアップを代行します。セットアップが終わり次第送り返します
-* 使い方に関しては、https://pbm-cloud.jiikko.com ですべて完結するので、Linuxの知識は不要です
-* 何か問題が起きた時はリモートログインでサポート可能です
-* 希望者はDiscordの `kawagu#7684` にDMを送ってください
-* お金はかかりません
-
 ## 開発を支援してくれる人を募集しています
 * https://jiikko.fanbox.cc/
 * procon_bypassの運営・開発・サーバー費用に充てさせていただきます。また、問い合わせに優先して対応します。

data/benchmarks/README.md

@@ -0,0 +1,6 @@
+# benchmark
+## Processing
+プロコンから読み取ったバイナリを加工する処理
+
+* `ruby benchmarks/bypass.rb`
+* `bundle execc stackprof tmp/stackprof.dump`

data/benchmarks/bypass.rb

@@ -0,0 +1,11 @@
+require "stackprof"
+require "procon_bypass_man"
+
+# no action
+raw_binary = ["30f28100800078c77448287509550274ff131029001b0022005a0271ff191028001e00210064027cff1410280020002100000000000000000000000000000000"].pack("H*")
+binary = ProconBypassMan::Domains::InboundProconBinary.new(binary: raw_binary)
+StackProf.run(mode: :cpu, out: 'tmp/stackprof.dump') do
+  10000.times do
+    ProconBypassMan::Processor.new(binary).process
+  end
+end

data/docs/getting_started.md

@@ -3,17 +3,16 @@
 * [はじめに](#はじめに)
     * [procon_bypass_manで解決したいこと](#procon_bypass_manで解決したいこと)
     * [procon_bypass_manでできること](#procon_bypass_manでできること)
-* [セットアップを代行できます](#セットアップを代行できます)
 * [セットアップ](#セットアップ)
   * [ラズベリーパイのセットアップ](#ラズベリーパイのセットアップ)
   * [procon_bypass_manのインストール](#procon_bypass_manのインストール)
       * [pbmenvを使う方法](#pbmenvを使う方法)
       * [pbmenvを使わない方法](#pbmenvを使わない方法)
+* [RaspberryPiの給電について](#RaspberryPiの給電について)
 * [普段使いをするためのセットアップ](#普段使いをするためのセットアップ)
 * [レイヤー](#レイヤー)
 * [マクロ](#マクロ)
 * [左スティックの感度調整](#左スティックの感度調整)
-* [入力表示](#入力表示)
 * [設定ファイルの書き方](#設定ファイルの書き方)
 * プラグインの書き方
 * [設定ファイルの書き方がわからない、エラーが起きるとき](#設定ファイルの書き方がわからない、エラーが起きるとき)
@@ -43,15 +42,10 @@
   * [設定方法](/docs/setting/left-analogstick-cap.md)
 * WEBから設定状態の閲覧・反映
 * ボタンリマップ
+<!--
 * 入力表示
   * https://github.com/splaplapla/switch-procon-input-viewer
-
-## セットアップを代行できます
-* Raspberry Pi4を開発者である私の家に配送してくれれば、セットアップを代行します。セットアップが終わり次第送り返します
-* 使い方に関しては、https://pbm-cloud.jiikko.com ですべて完結するので、Linuxの知識は不要です
-* 何か問題が起きた時はリモートログインでサポート可能です
-* 希望者はDiscordの `kawagu#7684` にDMを送ってください
-* お金はかかりません
+-->
 
 ## セットアップ
 ### ラズベリーパイのセットアップ
@@ -123,6 +117,14 @@ wget https://raw.githubusercontent.com/jiikko/procon_bypass_man_sample/master/se
 sudo /home/pi/.rbenv/versions/3.0.1/bin/ruby app.rb
 ```
 
+## RaspberryPiの給電について
+Raspberry Piの状態によっては、Switchと接続しているときに、Raspberry Piが電力不足になるようで動作が不安定になることがあります。  
+不安定になるようであれば、Switch以外からも給電してみてください。詳細には言及しませんが、主な給電方法には、以下があります。
+
+* GPIO端子
+* セルフパワーUSBハブ
+* PoE
+
 ## 普段使いをするためのセットアップ
 
 procon_bypass_manを起動するだけでプロコンと接続ができるようになったら、Raspberry Piを起動したときにprocon_bypass_manも自動起動するように設定しましょう。  
@@ -215,9 +217,11 @@ open_macro :sokuwari, steps: [:toggle_r_for_0_2sec, :toggle_thumbr_for_0_14sec,
 ## 左スティックの感度調整
 [左スティックの感度調整](/docs/setting/left-analogstick-cap.md)
 
+<!--
 ## 入力表示
 * 使い方は https://github.com/splaplapla/switch-procon-input-viewer を参照してください。
 * https://zenn.dev/jiikko/articles/2ef0ccbdfe0fe7 に技術的な解説を書きました
+-->
 
 ## 設定ファイルの書き方
 設定ファイルは、ymlフォーマットに埋め込まれたRubyスクリプトで記述します。  

data/docs/setting/left-analogstick-cap.md

@@ -54,7 +54,7 @@ setting: |-
   prefix_keys_for_changing_layer [:zr, :zl, :l]
   set_neutral_position 2100, 2000
 
-  layer :up, mode: :manual do
+  layer :up do
     left_analog_stick_cap cap: 1100, if_pressed: [:zl, :a], force_neutral: :a
   end
 ```

data/docs/setting/splatoon3_macro_dasei_cancel.md

@@ -3,6 +3,13 @@
 * 本マクロは実験段階で、オプション名などの仕様が変更される可能性が高いです
 * procon_bypass_man: 0.3.3以上が必要です
 
+
+
+https://user-images.githubusercontent.com/1664497/210229394-493a7ad4-b4cf-4dfd-87da-2398494e00fd.mp4
+
+
+
+
 ## 1. install_macro_pluginでマクロを有効化します
 * `setting.yml` に`install_macro_plugin ProconBypassMan::Plugin::Splatoon3::Macro::DaseiCancel` と書きます
 * これを記述することで、layer内でmacroを呼び出せるようになります

data/docs/setting/splatoon3_macro_forward_ikarole.md

@@ -26,7 +26,7 @@ version: 1.0
 setting: |-
   prefix_keys_for_changing_layer [:zr, :zl, :l]
 
-  install_macro_plugin(ProconBypassMan::Plugin::Splatoon2::Macro::ForwardIkarole)
+  install_macro_plugin(ProconBypassMan::Plugin::Splatoon3::Macro::ForwardIkarole)
 
   layer :up do
     macro ProconBypassMan::Plugin::Splatoon3::Macro::ForwardIkarole, if_pressed: [:thumbl], force_neutral: []

data/docs/setting/splatoon3_rotation_left_stick.md

@@ -0,0 +1,20 @@
+# スプラトゥーン3: 左スティック1回転 マクロの設定方法
+
+* procon_bypass_man: 0.3.4以上が必要です
+* このマクロは、左スティックを高速に1回転をします
+
+
+https://user-images.githubusercontent.com/1664497/205416889-d458668e-ab46-4867-890c-ce32ff467128.mp4
+
+
+## 設定例
+```yaml
+version: 1.0
+setting: |-
+  prefix_keys_for_changing_layer [:zr, :zl, :l]
+  install_macro_plugin(ProconBypassMan::Plugin::Splatoon3::Macro::RotationLeftStick)
+
+  layer :up do
+    macro ProconBypassMan::Plugin::Splatoon3::Macro::RotationLeftStick, if_pressed: [:left]
+  end
+```

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

@@ -0,0 +1,11 @@
+class ProconBypassMan::ReportWarningJob < ProconBypassMan::ReportEventBaseJob
+  extend ProconBypassMan::HasExternalApiSetting
+
+  # @param [String] body
+  def self.perform(body)
+    ProconBypassMan::ReportHttpClient.new(
+      path: path,
+      server: api_server,
+    ).post(body: body, event_type: :warn)
+  end
+end

data/lib/procon_bypass_man/background.rb

@@ -9,6 +9,7 @@ require "procon_bypass_man/background/jobs/report_reload_config_job"
 require "procon_bypass_man/background/jobs/report_error_reload_config_job"
 require "procon_bypass_man/background/jobs/report_load_config_job"
 require "procon_bypass_man/background/jobs/report_error_job"
+require "procon_bypass_man/background/jobs/report_warning_job"
 require "procon_bypass_man/background/jobs/report_completed_upgrade_pbm_job"
 require "procon_bypass_man/background/jobs/report_procon_performance_measurements_job"
 require "procon_bypass_man/background/jobs/sync_device_stats_job"

data/lib/procon_bypass_man/buttons_setting_configuration/loader.rb

@@ -1,4 +1,5 @@
 require "procon_bypass_man/buttons_setting_configuration/param_normalizer"
+require "procon_bypass_man/buttons_setting_configuration/metadata_loader"
 
 module ProconBypassMan
   class ButtonsSettingConfiguration
@@ -9,6 +10,11 @@ module ProconBypassMan
       def self.load(setting_path: )
         ProconBypassMan::ButtonsSettingConfiguration.instance.setting_path = setting_path
 
+        metadata_loader = ProconBypassMan::ButtonsSettingConfiguration::MetadataLoader.load(setting_path: setting_path)
+        if(Gem::Version.new(metadata_loader.required_pbm_version) >= Gem::Version.new(ProconBypassMan::VERSION))
+          ProconBypassMan::SendErrorCommand.execute(error: '起動中のPBMが設定ファイルのバージョンを満たしていません。設定ファイルが意図した通り動かない可能性があります。PBMのバージョンをあげてください。')
+        end
+
         ProconBypassMan::ButtonsSettingConfiguration.switch_new_context(:validation) do |new_instance|
           yaml = YAML.load_file(setting_path) or raise "読み込みに失敗しました"
           new_instance.instance_eval(yaml["setting"])

data/lib/procon_bypass_man/buttons_setting_configuration/metadata_loader.rb

@@ -0,0 +1,27 @@
+module ProconBypassMan
+  class ButtonsSettingConfiguration
+    class MetadataLoader
+      EMPTY_VERSION = '0.0.0'
+
+      # @param [String] setting_path
+      # @return [MetadataLoader]
+      def self.load(setting_path: )
+        self.new(setting_path)
+      end
+
+      # @param [String] setting_path
+      def initialize(setting_path)
+        content = File.read(setting_path)
+        if(matched = content.match(/metadata-required_pbm_version: ([\d.]+)/))
+          @required_pbm_version = matched[1]
+        end
+      end
+
+      # @return [String]
+      def required_pbm_version
+        return EMPTY_VERSION unless defined?(@required_pbm_version)
+        return @required_pbm_version if @required_pbm_version
+      end
+    end
+  end
+end

data/lib/procon_bypass_man/buttons_setting_configuration/validator.rb

@@ -12,7 +12,7 @@ module ProconBypassMan
       def valid?
         @errors = Hash.new {|h,k| h[k] = [] }
 
-        validate_require_prefix_keys
+        warn_blank_prefix_keys
         validate_config_of_button_lonely
         validate_verify_button_existence
         validate_flip_and_remap_are_hate_each_other
@@ -56,10 +56,10 @@ module ProconBypassMan
         end
       end
 
-      def validate_require_prefix_keys
+      # @return [void]
+      def warn_blank_prefix_keys
         if @prefix_keys.empty?
-          @errors[:prefix_keys] ||= []
-          @errors[:prefix_keys] << "prefix_keys_for_changing_layerに値が入っていません。"
+          ProconBypassMan.logger.warn "prefix_keys_for_changing_layerに値が入っていません。"
         end
       end
 

data/lib/procon_bypass_man/bypass/bypass_command.rb

@@ -1,6 +1,5 @@
+# 子プロセスで実行する
 class ProconBypassMan::BypassCommand
-  include ProconBypassMan::SignalHandler
-
   module WILL_TERMINATE_TOKEN
     TERMINATE = :terminate
     RESTART = :restart
@@ -25,7 +24,7 @@ class ProconBypassMan::BypassCommand
 
     # gadget => procon
     # 遅くていい
-    ProconBypassMan.logger.info "Thread1を起動します"
+    ProconBypassMan.logger.info "[BYPASS] Thread1を起動します"
 
     cycle_sleep = ProconBypassMan::CycleSleep.new(cycle_interval: 1, execution_cycle: ProconBypassMan.config.bypass_mode.gadget_to_procon_interval)
 
@@ -46,14 +45,14 @@ class ProconBypassMan::BypassCommand
         Process.kill "TERM", Process.ppid
       end
 
-      ProconBypassMan.logger.info "Thread1を終了します"
+      ProconBypassMan.logger.info "[BYPASS] Thread1を終了します"
     end
 
     # procon => gadget
     # シビア
     t2 = Thread.new do
       bypass = ProconBypassMan::Bypass::ProconToSwitch.new(gadget: @gadget, procon: @procon)
-      process = BlueGreenProcess.new(worker_instance: bypass, max_work: 550)
+      process = BlueGreenProcess.new(worker_instance: bypass, max_work: 1000)
       loop do
         if $will_terminate_token
           if $will_terminate_token == WILL_TERMINATE_TOKEN::TERMINATE
@@ -66,8 +65,8 @@ class ProconBypassMan::BypassCommand
         process.work
 
         process_switching_time_before_work = BlueGreenProcess.performance.process_switching_time_before_work
-        if process_switching_time_before_work > 0.1
-          ProconBypassMan.logger.info("slow process_switching_time_before_work: #{process_switching_time_before_work}")
+        if process_switching_time_before_work > 0.2
+          ProconBypassMan::SendWarningCommand.execute(warning: "[PERFORMANCE] BlueGreenProcess.performance.process_switching_time_before_work: #{process_switching_time_before_work}", stdout: false)
         end
 
       rescue EOFError => e
@@ -81,25 +80,49 @@ class ProconBypassMan::BypassCommand
         process.shutdown
         break
       end
+
+      ProconBypassMan.logger.info "[BYPASS] Thread2を終了します"
     end
 
-    ProconBypassMan.logger.info "子プロセスでgraceful shutdownの準備ができました"
+    ProconBypassMan.logger.info "BYPASSプロセスでgraceful shutdownの準備ができました"
     begin
+      # TODO: 本当はいらないんだけど、なぜか反映されないのでここでも設定する
+      BlueGreenProcess.config.logger = ProconBypassMan.logger
+
       while(readable_io = IO.select([self_read]))
         signal = readable_io.first[0].gets.strip
-        handle_signal(signal)
+        case signal
+        when 'USR2'
+          ProconBypassMan.logger.debug "[BYPASS] BYPASSプロセスでUSR2シグナルを受け取りました"
+          raise ProconBypassMan::InterruptForRestart
+        when 'TERM'
+          ProconBypassMan.logger.debug "[BYPASS] BYPASSプロセスでTERMシグナルを受け取りました"
+          raise Interrupt
+        when 'INT'
+          ProconBypassMan.logger.debug "[BYPASS] BYPASSプロセスでINTシグナルを無視します"
+        end
       end
     rescue ProconBypassMan::InterruptForRestart
+      ProconBypassMan.logger.info "[BYPASS] BYPASSプロセスの終了処理を開始します"
       $will_terminate_token = WILL_TERMINATE_TOKEN::RESTART
+      BlueGreenProcess.terminate_workers_immediately
       [t1, t2].each(&:join)
+      ProconBypassMan.logger.info "[BYPASS] BYPASSプロセス内のThreadsが停止しました"
       @gadget&.close
       @procon&.close
+      ProconBypassMan.logger.info "[BYPASS] BYPASSプロセスを終了します"
+      DRb.stop_service if defined?(DRb)
       exit! 1 # child processなのでexitしていい
     rescue Interrupt
+      ProconBypassMan.logger.info "[BYPASS] BYPASSプロセスの終了処理を開始します"
       $will_terminate_token = WILL_TERMINATE_TOKEN::TERMINATE
+      BlueGreenProcess.terminate_workers_immediately
       [t1, t2].each(&:join)
+      ProconBypassMan.logger.info "[BYPASS] BYPASSプロセス内のThreadsが停止しました"
       @gadget&.close
       @procon&.close
+      ProconBypassMan.logger.info "[BYPASS] BYPASSプロセスを終了します"
+      DRb.stop_service if defined?(DRb)
       exit! 1 # child processなのでexitしていい
     end
   end

data/lib/procon_bypass_man/bypass/procon_to_switch.rb

@@ -29,18 +29,18 @@ class ProconBypassMan::Bypass::ProconToSwitch
       self.bypass_value = ProconBypassMan::Bypass::BypassValue.new(nil)
 
       next(run_callbacks(:work) {
-        next(false) if $will_terminate_token
+        next(false) if will_terminate?
 
         raw_output = nil
         measurement.record_read_time do
           begin
-            return(false) if $will_terminate_token
+            return(false) if will_terminate?
             raw_output = self.procon.read_nonblock(64)
           rescue IO::EAGAINWaitReadable
             sleep(0.002)
             retry
           rescue Errno::EIO, Errno::ENODEV, Errno::EPROTO, IOError, Errno::ESHUTDOWN, Errno::ETIMEDOUT => e
-            return(false) if $will_terminate_token
+            return(false) if will_terminate?
             raise
           end
 
@@ -52,7 +52,7 @@ class ProconBypassMan::Bypass::ProconToSwitch
             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_token # rubocop:disable Lint/NoReturnInBeginEndBlocks
+                return(false) if will_terminate? # rubocop:disable Lint/NoReturnInBeginEndBlocks
                 binary = ::ProconBypassMan::Procon::Rumbler.monitor do
                   ProconBypassMan::Processor.new(bypass_value.binary).process
                 end
@@ -69,11 +69,11 @@ class ProconBypassMan::Bypass::ProconToSwitch
 
                 next(true)
               rescue IO::EAGAINWaitReadable
-                return(false) if $will_terminate_token # rubocop:disable Lint/NoReturnInBeginEndBlocks
+                return(false) if will_terminate? # rubocop:disable Lint/NoReturnInBeginEndBlocks
                 measurement.record_write_error
                 raise CouldNotWriteToSwitchError
               rescue Errno::EIO, Errno::ENODEV, Errno::EPROTO, IOError, Errno::ESHUTDOWN, Errno::ETIMEDOUT => e
-                return(false) if $will_terminate_token # rubocop:disable Lint/NoReturnInBeginEndBlocks
+                return(false) if will_terminate? # rubocop:disable Lint/NoReturnInBeginEndBlocks
                 raise
               end
             end
@@ -89,7 +89,7 @@ class ProconBypassMan::Bypass::ProconToSwitch
 
   # @return [void]
   def direct_connect_switch_via_bluetooth
-    ProconBypassMan.logger.debug { "direct_connect_switch_via_bluetooth!" }
+    ProconBypassMan.logger.debug { "[BYPASS] プロコンとSwitchを無線接続へ切り替えます" }
     self.procon.write_nonblock(["010500000000000000003800"].pack("H*")) # home led off
     self.procon.write_nonblock(["010600000000000000003800"].pack("H*")) # home led off
     self.procon.write_nonblock(["010700000000000000003800"].pack("H*")) # home led off
@@ -107,4 +107,9 @@ class ProconBypassMan::Bypass::ProconToSwitch
 
     ProconBypassMan.logger.debug { "<<< #{bypass_value.to_text}" }
   end
+
+  # @return [Boolean]
+  def will_terminate?
+    $will_terminate_token
+  end
 end

data/lib/procon_bypass_man/bypass/switch_to_procon.rb

@@ -58,9 +58,9 @@ class ProconBypassMan::Bypass::SwitchToProcon
     if ProconBypassMan.config.verbose_bypass_log
       ProconBypassMan.logger.debug { ">>> #{bypass_value.to_text}" }
     else
-      ProconBypassMan.cache.fetch key: 'bypass_log', expires_in: 1 do
-        ProconBypassMan.logger.debug { ">>> #{bypass_value.to_text}" }
-      end
+      # ProconBypassMan.cache.fetch key: 'bypass_log', expires_in: 1 do
+      #   ProconBypassMan.logger.debug { ">>> #{bypass_value.to_text}" }
+      # end
     end
   end
 end

data/lib/procon_bypass_man/commands/send_warning_command.rb

@@ -0,0 +1,18 @@
+class ProconBypassMan::SendWarningCommand
+  # @param [String, Hash, Exception] warning
+  # @return [void]
+  def self.execute(warning: , stdout: true)
+    body =
+      case warning
+      when String, Hash
+        warning
+      else
+        warning.full_message
+      end
+
+    ProconBypassMan.logger.warn body
+    puts body if stdout
+
+    ProconBypassMan::ReportWarningJob.perform(warning)
+  end
+end

data/lib/procon_bypass_man/commands.rb

@@ -4,3 +4,4 @@ require "procon_bypass_man/commands/write_session_id_command"
 require "procon_bypass_man/commands/write_device_id_command"
 require "procon_bypass_man/commands/send_reload_config_event_command"
 require "procon_bypass_man/commands/send_error_command"
+require "procon_bypass_man/commands/send_warning_command"

data/lib/procon_bypass_man/plugin/splatoon3/macro/rotation_left_stick.rb

@@ -0,0 +1,21 @@
+module ProconBypassMan
+  module Plugin
+    module Splatoon3
+      module Macro
+        module RotationLeftStick
+          def self.display_name
+            :rotation_left_stick
+          end
+
+          def self.steps
+            [:rotation_left_stick]
+          end
+
+          def self.description
+            '左スティックを1回転します'
+          end
+        end
+      end
+    end
+  end
+end

data/lib/procon_bypass_man/plugins.rb

@@ -18,6 +18,7 @@ require_relative "plugin/splatoon3/macro/sokuwari_for_splash_bomb"
 require_relative "plugin/splatoon3/macro/forward_ikarole"
 require_relative "plugin/splatoon3/macro/charge_tansan_bomb"
 require_relative "plugin/splatoon3/macro/dasei_cancel"
+require_relative "plugin/splatoon3/macro/rotation_left_stick"
 require_relative "plugin/splatoon3/mode/guruguru"
 
 module ProconBypassMan

data/lib/procon_bypass_man/procon/macro_builder.rb

@@ -95,7 +95,7 @@ class ProconBypassMan::Procon::MacroBuilder
     end
 
     # NOTE: マクロ構文で生成したいけど、スティックとボタン同時押しの構文が思いつかないので、ハードコードする
-    if /forward_ikarole1/ =~ step
+    if /^forward_ikarole1/ =~ step
       # NOTE: 0degはx: 1, y: 0, 90degはx: 0, y: 1, 180degはx: -1, y: 0.
       # NOTE: スティックを前方に倒している状態で270度回転させる
       for_forward_ikarole_steps = [
@@ -105,6 +105,11 @@ class ProconBypassMan::Procon::MacroBuilder
       return { steps: for_forward_ikarole_steps }
     end
 
+    if /^rotation_left_stick/ =~ step
+      roll_left_stick_steps = 0.step(359, 17).map { |x| ["tilt_left_stick_completely_to_#{x}deg".to_sym] }
+      return { steps: roll_left_stick_steps }
+    end
+
     if %r!^(pressing_|toggle_|shake_left_stick_)! =~ step && (subjects = step.scan(%r!pressing_[^_]+|shake_left_stick|toggle_[^_]+!)) && (match = step.match(%r!_for_([\d_]+)(sec)?\z!))
       if sec = match[1]
         return {

data/lib/procon_bypass_man/remote_pbm_action/change_pbm_version_action.rb

@@ -9,6 +9,7 @@ module ProconBypassMan
         Pbmenv.uninstall(pbm_version) # 途中でシャットダウンしてしまった、とか状態が途中の状態かもしれないので一旦消す
         Pbmenv.install(pbm_version, enable_pbm_cloud: true)
         Pbmenv.use(pbm_version)
+        Pbmenv.clean(10)
         ProconBypassMan.logger.info "#{pbm_version}へアップグレードしました"
         ProconBypassMan::ReportCompletedUpgradePbmJob.perform
         `reboot` # symlinkの参照先が変わるのでrebootする必要がある

data/lib/procon_bypass_man/runner.rb

@@ -1,6 +1,5 @@
+# フォアグラウンドで実行する
 class ProconBypassMan::Runner
-  include ProconBypassMan::SignalHandler
-
   def initialize(gadget: , procon: )
     @gadget = gadget
     @procon = procon
@@ -20,8 +19,9 @@ class ProconBypassMan::Runner
 
     loop do
       child_pid = Kernel.fork do
+        ProconBypassMan.logger.info "[BYPASS] BYPASSプロセスを起動します"
         $will_terminate_token = false
-        ProconBypassMan.after_fork_on_bypass_process
+        ProconBypassMan.run_on_after_fork_of_bypass_process
         ProconBypassMan::BypassCommand.new(gadget: @gadget, procon: @procon).execute # ここでblockingする
         next
       end
@@ -30,7 +30,13 @@ class ProconBypassMan::Runner
         # TODO 子プロセスが消滅した時に、メインプロセスは生き続けてしまい、何もできなくなる問題がある
         while(readable_io = IO.select([self_read]))
           signal = readable_io.first[0].gets.strip
-          handle_signal(signal)
+          ProconBypassMan.logger.debug "[BYPASS] MASTERプロセスで#{signal}シグナルを受け取りました"
+          case signal
+          when 'USR2'
+            raise ProconBypassMan::InterruptForRestart
+          when 'INT', 'TERM'
+            raise Interrupt
+          end
         end
       rescue ProconBypassMan::InterruptForRestart
         ProconBypassMan::PrintMessageCommand.execute(text: "設定ファイルの再読み込みを開始します")
@@ -40,17 +46,20 @@ class ProconBypassMan::Runner
           ProconBypassMan::ButtonsSettingConfiguration::Loader.reload_setting
           ProconBypassMan::SendReloadConfigEventCommand.execute
         rescue ProconBypassMan::CouldNotLoadConfigError => error
-          ProconBypassMan::SendErrorCommand.execute(error: "設定ファイルが不正です。再読み込みができませんでした")
+          ProconBypassMan::SendErrorCommand.execute(error: "[MASTER] 設定ファイルが不正です。再読み込みができませんでした")
           ProconBypassMan::ReportErrorReloadConfigJob.perform_async(error.message)
         end
-        ProconBypassMan::PrintMessageCommand.execute(text: "バイパス処理を再開します")
+        ProconBypassMan::PrintMessageCommand.execute(text: "[MASTER] バイパス処理を再開します")
       rescue Interrupt
         puts
-        ProconBypassMan::PrintMessageCommand.execute(text: "処理を終了します")
+        ProconBypassMan::PrintMessageCommand.execute(text: "[MASTER] BYPASSプロセスにTERMシグナルを送信します")
         Process.kill("TERM", child_pid)
+        ProconBypassMan::PrintMessageCommand.execute(text: "[MASTER] BYPASSプロセスの終了を待ちます")
         Process.wait
         break
       end
     end
+
+    ProconBypassMan::PrintMessageCommand.execute(text: "[MASTER] メインプロセスを終了します")
   end
 end

data/lib/procon_bypass_man/version.rb

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

data/lib/procon_bypass_man.rb

@@ -20,7 +20,6 @@ require_relative "procon_bypass_man/device_connection"
 require_relative "procon_bypass_man/support/usb_device_controller"
 require_relative "procon_bypass_man/support/device_procon_finder"
 require_relative "procon_bypass_man/support/device_mouse_finder"
-require_relative "procon_bypass_man/support/signal_handler"
 require_relative "procon_bypass_man/support/callbacks"
 require_relative "procon_bypass_man/support/yaml_loader"
 require_relative "procon_bypass_man/support/yaml_writer"
@@ -39,7 +38,6 @@ require_relative "procon_bypass_man/support/analog_stick_hypotenuse_tilting_powe
 require_relative "procon_bypass_man/support/never_exit_accidentally"
 require_relative "procon_bypass_man/support/cycle_sleep"
 require_relative "procon_bypass_man/support/can_over_process"
-require_relative "procon_bypass_man/support/gc"
 require_relative "procon_bypass_man/support/retryable"
 require_relative "procon_bypass_man/support/renice_command"
 require_relative "procon_bypass_man/procon_display"
@@ -182,14 +180,16 @@ module ProconBypassMan
   end
 
   # @return [void]
-  def self.after_fork_on_bypass_process
+  def self.run_on_after_fork_of_bypass_process
     ProconBypassMan::ReniceCommand.change_priority(to: :high, pid: $$)
     ::GC.start
-    DRb.start_service if defined?(DRb)
-    # GC対策することによって一時的に削除した機能
+    # GC対策することによって一時的に削除した機能. 後で有効にしたい
     # ProconBypassMan::ProconDisplay::Server.start!
-
     DRb.start_service if defined?(DRb)
+
+    # for libs setting
+    BlueGreenProcess.config.logger = ProconBypassMan.logger
+
     BlueGreenProcess.configure do |config|
       config.after_fork = -> {
         DRb.start_service if defined?(DRb)

data/procon_bypass_man.gemspec

@@ -28,11 +28,10 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
 
   # Uncomment to register a new dependency of your gem
-  spec.add_dependency "pbmenv", ">= 0.1.9" # enable_pbm_cloud連携機能でPbmenv.installを使っている.
+  spec.add_dependency "pbmenv", ">= 0.1.11" # enable_pbm_cloud連携機能でPbmenv.install, .cleanを使っている.
   spec.add_dependency "action_cable_client"
   spec.add_dependency "sorted_set"
-  spec.add_dependency "blue_green_process"
-  # spec.add_dependency "blue_green_process"
+  spec.add_dependency "blue_green_process", '0.1.4.2'
 
   # For more information and examples about making a new gem, checkout our
   # guide at: https://bundler.io/guides/creating_gem.html

data/project_template/app.rb

@@ -12,7 +12,7 @@ begin
   gemfile do
     source 'https://rubygems.org'
     git_source(:github) {|repo_name| "https://github.com/#{repo_name}" }
-    gem 'procon_bypass_man', '0.3.3.1'
+    gem 'procon_bypass_man', '0.3.5'
   end
 rescue Bundler::Source::Git::GitCommandError => e
   retry_count_on_git_command_error = retry_count_on_git_command_error + 1

data/project_template/app.rb.erb

@@ -12,7 +12,7 @@ begin
   gemfile do
     source 'https://rubygems.org'
     git_source(:github) {|repo_name| "https://github.com/#{repo_name}" }
-    gem 'procon_bypass_man', '0.3.3.1'
+    gem 'procon_bypass_man', '0.3.5'
   end
 rescue Bundler::Source::Git::GitCommandError => e
   retry_count_on_git_command_error = retry_count_on_git_command_error + 1

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: procon_bypass_man
 version: !ruby/object:Gem::Version
-  version: 0.3.3.1
+  version: 0.3.5
 platform: ruby
 authors:
 - jiikko
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-11-13 00:00:00.000000000 Z
+date: 2023-02-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pbmenv
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.1.9
+        version: 0.1.11
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.1.9
+        version: 0.1.11
 - !ruby/object:Gem::Dependency
   name: action_cable_client
   requirement: !ruby/object:Gem::Requirement
@@ -56,16 +56,16 @@ dependencies:
   name: blue_green_process
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.1.4.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.1.4.2
 description: A programmable converter for Nintendo Switch Pro Controller
 email:
 - n905i.1214@gmail.com
@@ -88,6 +88,8 @@ files:
 - README.md
 - Rakefile
 - Steepfile
+- benchmarks/README.md
+- benchmarks/bypass.rb
 - bin/console
 - bin/dev_api_server.rb
 - bin/generate_default_app
@@ -104,6 +106,7 @@ files:
 - docs/setting/splatoon3_macro_dasei_cancel.md
 - docs/setting/splatoon3_macro_forward_ikarole.md
 - docs/setting/splatoon3_recommended_setting.md
+- docs/setting/splatoon3_rotation_left_stick.md
 - docs/setup_raspi.md
 - docs/setup_raspi.mitamae.rb
 - docs/setup_raspi_by_mitamae.md
@@ -126,10 +129,12 @@ files:
 - lib/procon_bypass_man/background/jobs/report_procon_performance_measurements_job.rb
 - lib/procon_bypass_man/background/jobs/report_reload_config_job.rb
 - lib/procon_bypass_man/background/jobs/report_start_reboot_job.rb
+- lib/procon_bypass_man/background/jobs/report_warning_job.rb
 - lib/procon_bypass_man/background/jobs/sync_device_stats_job.rb
 - lib/procon_bypass_man/buttons_setting_configuration.rb
 - lib/procon_bypass_man/buttons_setting_configuration/layer.rb
 - lib/procon_bypass_man/buttons_setting_configuration/loader.rb
+- lib/procon_bypass_man/buttons_setting_configuration/metadata_loader.rb
 - lib/procon_bypass_man/buttons_setting_configuration/param_normalizer.rb
 - lib/procon_bypass_man/buttons_setting_configuration/param_normalizer/button.rb
 - lib/procon_bypass_man/buttons_setting_configuration/param_normalizer/button_list.rb
@@ -150,6 +155,7 @@ files:
 - lib/procon_bypass_man/commands/print_message_command.rb
 - lib/procon_bypass_man/commands/send_error_command.rb
 - lib/procon_bypass_man/commands/send_reload_config_event_command.rb
+- lib/procon_bypass_man/commands/send_warning_command.rb
 - lib/procon_bypass_man/commands/write_device_id_command.rb
 - lib/procon_bypass_man/commands/write_session_id_command.rb
 - lib/procon_bypass_man/configuration.rb
@@ -183,6 +189,7 @@ files:
 - lib/procon_bypass_man/plugin/splatoon3/macro/jump_to_left_key.rb
 - lib/procon_bypass_man/plugin/splatoon3/macro/jump_to_right_key.rb
 - lib/procon_bypass_man/plugin/splatoon3/macro/jump_to_up_key.rb
+- lib/procon_bypass_man/plugin/splatoon3/macro/rotation_left_stick.rb
 - lib/procon_bypass_man/plugin/splatoon3/macro/sokuwari_for_splash_bomb.rb
 - lib/procon_bypass_man/plugin/splatoon3/mode/guruguru.rb
 - lib/procon_bypass_man/plugin/splatoon3/version.rb
@@ -256,7 +263,6 @@ files:
 - lib/procon_bypass_man/support/cycle_sleep.rb
 - lib/procon_bypass_man/support/device_mouse_finder.rb
 - lib/procon_bypass_man/support/device_procon_finder.rb
-- lib/procon_bypass_man/support/gc.rb
 - lib/procon_bypass_man/support/http_client.rb
 - lib/procon_bypass_man/support/load_agv.rb
 - lib/procon_bypass_man/support/never_exit_accidentally.rb
@@ -268,7 +274,6 @@ files:
 - lib/procon_bypass_man/support/retryable.rb
 - lib/procon_bypass_man/support/safe_timeout.rb
 - lib/procon_bypass_man/support/send_device_stats_http_client.rb
-- lib/procon_bypass_man/support/signal_handler.rb
 - lib/procon_bypass_man/support/update_remote_pbm_action_status_http_client.rb
 - lib/procon_bypass_man/support/uptime.rb
 - lib/procon_bypass_man/support/usb_device_controller.rb

data/lib/procon_bypass_man/support/gc.rb

@@ -1,8 +0,0 @@
-class ProconBypassMan::GC
-  def self.stop_gc_in(&block)
-    ::GC.disable
-    result = block.call
-    ::GC.enable
-    return result
-  end
-end

data/lib/procon_bypass_man/support/signal_handler.rb

@@ -1,11 +0,0 @@
-module ProconBypassMan::SignalHandler
-  def handle_signal(sig)
-    ProconBypassMan.logger.info "#{$$}で#{sig}を受け取りました"
-    case sig
-    when 'USR2'
-      raise ProconBypassMan::InterruptForRestart
-    when 'INT', 'TERM'
-      raise Interrupt
-    end
-  end
-end