fusuma-plugin-tap 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2bedc73a6f30eada228504ea4b0c09b7c8cfb274a67764f6bcc93df88d63439d
-  data.tar.gz: 9b00f18217502e62dc83776231bf6de7ef5f7705f04178c67f4dd1f569675a77
+  metadata.gz: 0b2d9fc25b57d8e8475739f63334e1996b382790d2f69934cfc4af3a191cf183
+  data.tar.gz: 9b7b628758a30e219c902b7e9a62675b415251277c078b2e40b794f7c7c2b1aa
 SHA512:
-  metadata.gz: 11ea664f6f1ca08ef08fc0ef759634066a7f813cf5b43502294a034e96349e58fddbce4e223d3ac858c847d9a508a91335b4f262bc25b992e91f3559a9591a82
-  data.tar.gz: a0db20541d07c7542b7c38b8c6f34cd61160dce875dfd103251446dc80cf173fd6fec1e20cc61d476bf912af7b6c2ae024e4e2e761577c1e87165f157c86cc56
+  metadata.gz: 9d6e782d13635368460a1f6f2c6ae04a3f09aec5e4d22b9ef8f9f12030c81570fb0cd174573e0aa5c25b9022163c8d43edd6de84c378cd81c75aaa58c7d31877
+  data.tar.gz: ad374930be4394b1bfc503024115b1dd6181eb3d597ba02f4095002fed3fbcca8a60d8ccfa04b11034f5ba42532448f3ddf63d3cff67085edc62dbc8da35af15

data/.rubocop.yml

@@ -18,6 +18,11 @@ Metrics/MethodLength:
     - 'lib/fusuma/plugin/parsers/tap_parser.rb'
     - 'lib/fusuma/plugin/parsers/tap_detector.rb'
 
+Metrics/ClassLength:
+  Exclude:
+    - 'lib/fusuma/plugin/parsers/tap_parser.rb'
+    - 'lib/fusuma/plugin/parsers/tap_detector.rb'
+
 Layout/LineLength:
   Max: 100
   Exclude:

data/.rubocop_todo.yml

@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2020-05-10 21:28:27 +0900 using RuboCop version 0.82.0.
+# on 2020-07-05 15:49:25 UTC using RuboCop version 0.86.0.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
@@ -16,7 +16,7 @@ Gemspec/RequiredRubyVersion:
 # Offense count: 5
 # Configuration parameters: IgnoredMethods.
 Metrics/AbcSize:
-  Max: 40
+  Max: 42
 
 # Offense count: 1
 # Configuration parameters: CountComments, ExcludedMethods.
@@ -27,17 +27,17 @@ Metrics/BlockLength:
 # Offense count: 3
 # Configuration parameters: IgnoredMethods.
 Metrics/CyclomaticComplexity:
-  Max: 24
+  Max: 25
 
 # Offense count: 5
 # Configuration parameters: CountComments, ExcludedMethods.
 Metrics/MethodLength:
   Max: 45
 
-# Offense count: 3
+# Offense count: 2
 # Configuration parameters: IgnoredMethods.
 Metrics/PerceivedComplexity:
-  Max: 11
+  Max: 13
 
 # Offense count: 1
 Style/Documentation:

data/CHANGELOG.md

@@ -1,5 +1,45 @@
 # Changelog
 
+## [v0.4.0.pre2](https://github.com/iberianpig/fusuma-plugin-tap/tree/v0.4.0.pre2) (2021-03-29)
+
+[Full Changelog](https://github.com/iberianpig/fusuma-plugin-tap/compare/v0.4.0.pre...v0.4.0.pre2)
+
+## [v0.4.0.pre](https://github.com/iberianpig/fusuma-plugin-tap/tree/v0.4.0.pre) (2020-11-09)
+
+[Full Changelog](https://github.com/iberianpig/fusuma-plugin-tap/compare/v0.3.2...v0.4.0.pre)
+
+## [v0.3.2](https://github.com/iberianpig/fusuma-plugin-tap/tree/v0.3.2) (2020-08-13)
+
+[Full Changelog](https://github.com/iberianpig/fusuma-plugin-tap/compare/v0.3.1...v0.3.2)
+
+**Fixed bugs:**
+
+- Fix Parser when detecting a false 4 finger tap after a pinch gesture [\#5](https://github.com/iberianpig/fusuma-plugin-tap/pull/5) ([iberianpig](https://github.com/iberianpig))
+
+**Closed issues:**
+
+- tap 4 not registered [\#4](https://github.com/iberianpig/fusuma-plugin-tap/issues/4)
+
+**Merged pull requests:**
+
+- test\(tap\_parser\): reproduce the bug that 4 finger tap after detecting palm [\#8](https://github.com/iberianpig/fusuma-plugin-tap/pull/8) ([iberianpig](https://github.com/iberianpig))
+
+## [v0.3.1](https://github.com/iberianpig/fusuma-plugin-tap/tree/v0.3.1) (2020-05-15)
+
+[Full Changelog](https://github.com/iberianpig/fusuma-plugin-tap/compare/v0.3.0...v0.3.1)
+
+**Closed issues:**
+
+- Reduce false positives with Hold gesture [\#2](https://github.com/iberianpig/fusuma-plugin-tap/issues/2)
+
+## [v0.3.0](https://github.com/iberianpig/fusuma-plugin-tap/tree/v0.3.0) (2020-05-10)
+
+[Full Changelog](https://github.com/iberianpig/fusuma-plugin-tap/compare/v0.2.0...v0.3.0)
+
+**Closed issues:**
+
+- 4 finger tap / hold is not being registered [\#1](https://github.com/iberianpig/fusuma-plugin-tap/issues/1)
+
 ## [v0.2.0](https://github.com/iberianpig/fusuma-plugin-tap/tree/v0.2.0) (2020-05-05)
 
 [Full Changelog](https://github.com/iberianpig/fusuma-plugin-tap/compare/v0.1.3...v0.2.0)

data/Makefile

@@ -3,11 +3,22 @@
 all: help
 
 build: ## build libinput with LIBINPUT_VERSION
-	cd $(LIBINPUT_SOURCE_DIR) && \
+	cd $(LIBINPUT_REPO) && \
+		git fetch && \
 		git checkout $(LIBINPUT_VERSION) && \
-		meson --prefix=/usr builddir/ -Dlibwacom=false -Ddebug-gui=false -Dtests=false -Ddocumentation=false --reconfigure && \
-		ninja -C builddir/
-	$(LIBINPUT_LIST_DEVICES) --version
+		mkdir -p builddir/$(LIBINPUT_VERSION)/ && \
+		meson --prefix=/usr builddir/$(LIBINPUT_VERSION)/ -Dlibwacom=false -Ddebug-gui=false -Dtests=false -Ddocumentation=false && \
+		ninja -C builddir/$(LIBINPUT_VERSION)/
+	$(LIBINPUT_REPO)/builddir/$(LIBINPUT_VERSION)/libinput-list-devices --version
+
+version: ## libinput list-devces --version
+	$(LIBINPUT_REPO)/builddir/$(LIBINPUT_VERSION)/libinput-list-devices --version
+
+list-devices: ## libinput list-devices
+	$(LIBINPUT_REPO)/builddir/$(LIBINPUT_VERSION)/libinput-list-devices
+
+debug-events: ## libinput debug-events
+	$(LIBINPUT_REPO)/builddir/$(LIBINPUT_VERSION)/libinput-debug-events --enable-tap --verbose
 
 help: ## show help
 	@grep -E '^[a-zA-Z0-9_-]+:.*?## .*$$' $(MAKEFILE_LIST) | sort | uniq | awk 'BEGIN {FS = ":.*?## "}; {printf "\033[36m%-30s\033[0m %s\n", $$1, $$2}'

data/README.md

@@ -62,6 +62,11 @@ plugin:
       verbose: true
 ```
 
+## Known issues
+* `libinput debug-events --verbose` outputs is unstable, so it may not work properly with some versions of libinput.
+* **libinput 1.10.4 contains a bug that suddenly stopped detecting taps.**
+  * ref: https://github.com/iberianpig/fusuma-plugin-tap/pull/8
+
 ## Development
 
 After checking out the repo, run `bin/setup` to install dependencies. Then, run `rake spec` to run the tests. You can also run `bin/console` for an interactive prompt that will allow you to experiment.

data/bin/libinput_debug_events_generator.rb

@@ -4,15 +4,16 @@
 require 'fileutils'
 require 'pathname'
 require 'fusuma/device'
+require 'fusuma/plugin/inputs/libinput_command_input'
 
 # Generate libinput generator
 class LibinputDebugEventsGenerator
   def initialize(evemu_record:)
     @evemu_record = evemu_record
     @gesture_name = Pathname.new(evemu_record).basename
-    @list_devices = ENV.fetch('LIBINPUT_LIST_DEVICES', 'libinput list-devices')
-    @debug_events = ENV.fetch('LIBINPUT_DEBUG_EVENTS', 'libinput debug-events')
-    @libinput_version = ENV.fetch('LIBINPUT_VERSION', 'libinput list-devices')
+    @list_devices = ENV.fetch('LIBINPUT_LIST_DEVICES')
+    @debug_events = ENV.fetch('LIBINPUT_DEBUG_EVENTS')
+    @libinput_version = ENV.fetch('LIBINPUT_VERSION')
   end
 
   def generate_debug_events
@@ -41,6 +42,10 @@ class LibinputDebugEventsGenerator
       rescue Timeout::Error
         puts 'cut out device infos'
       end
+      if `which evemu-play` == ''
+        warn 'evemu-play is not found'
+        exit 1
+      end
 
       command = "evemu-play /dev/input/#{device_id} < #{@evemu_record}"
 

data/fusuma-plugin-tap.gemspec

@@ -26,7 +26,7 @@ Gem::Specification.new do |spec|
 
   spec.required_ruby_version = '>= 2.3' # https://packages.ubuntu.com/search?keywords=ruby&searchon=names&exact=1&suite=all&section=main
 
-  spec.add_dependency 'fusuma', '~> 1.10'
+  spec.add_dependency 'fusuma', '~> 2.0.0'
 
   spec.add_development_dependency 'bundler'
   spec.add_development_dependency 'github_changelog_generator', '~> 1.14'

data/lib/fusuma/plugin/buffers/tap_buffer.rb

@@ -22,7 +22,7 @@ module Fusuma
         # @param event [Event]
         # @return [NilClass, TapBuffer]
         def buffer(event)
-          return unless (event&.tag == source) || (event&.tag == 'libinput_gesture_parser')
+          return unless event&.tag == source
 
           # NOTE: need to set `begin` event at first of buffer
           clear && return unless bufferable?(event)
@@ -56,12 +56,6 @@ module Fusuma
             else
               true
             end
-          when 'update' # libinput_gesture_parser
-            if empty?
-              false
-            else
-              true
-            end
           else
             raise "Unexpected status: #{event.record.status}"
           end

data/lib/fusuma/plugin/detectors/tap_detector.rb

@@ -7,6 +7,7 @@ module Fusuma
     module Detectors
       # Detect tap event
       class TapDetector < Detector
+        SOURCES = ['tap', 'gesture', 'timer']
         BUFFER_TYPE = 'tap'
         GESTURE_RECORD_TYPE = 'tap'
 
@@ -18,24 +19,27 @@ module Fusuma
         # @return [Event] if event is detected
         # @return [NilClass] if event is NOT detected
         def detect(buffers)
-          buffer = buffers.find { |b| b.type == BUFFER_TYPE }
+          tap_buffer = buffers.find { |b| b.type == BUFFER_TYPE }
           gesture_buffer = buffers.find { |b| b.type == 'gesture' }
+          timer_buffer = buffers.find { |b| b.type == 'timer' }
 
-          return if buffer.empty? || !gesture_buffer.empty? || moved?(buffer)
+          if tap_buffer.empty? || moved?(tap_buffer: tap_buffer, gesture_buffer: gesture_buffer)
+            return
+          end
 
-          holding_time = buffer.events.last.time - buffer.events.first.time
+          holding_time = calc_holding_time(tap_buffer: tap_buffer, timer_buffer: timer_buffer)
 
-          direction = if hold?(buffer, holding_time)
+          direction = if hold?(tap_buffer, holding_time)
                         'hold'
-                      elsif tap?(buffer, holding_time)
+                      elsif tap?(tap_buffer, holding_time)
                         'tap'
                       end
 
           return if direction.nil?
 
-          finger = buffer.finger
+          finger = tap_buffer.finger
 
-          buffer.clear
+          tap_buffer.clear # NOTE: Clear after detecting hold
 
           index = create_index(finger: finger, direction: direction)
 
@@ -71,8 +75,10 @@ module Fusuma
         end
 
         # @return [TrueClass, FalseClass]
-        def moved?(buffer)
-          buffer.events.any? { |e| e.record.status == 'move' }
+        def moved?(tap_buffer:, gesture_buffer:)
+          tap_buffer.events.any? { |e| e.record.status == 'move' } ||
+            # FIXME: Find good parameter for ignoring
+            gesture_buffer.events.count { |e| tap_buffer.events.first.time < e.time } > 5
         end
 
         # @return [TrueClass, FalseClass]
@@ -95,6 +101,16 @@ module Fusuma
           end
         end
 
+        def calc_holding_time(tap_buffer:, timer_buffer:)
+          last_time = if timer_buffer && !timer_buffer.empty? &&
+                         (tap_buffer.events.last.time < timer_buffer.events.last.time)
+                        timer_buffer.events.last.time
+                      else
+                        tap_buffer.events.last.time
+                      end
+          last_time - tap_buffer.events.first.time
+        end
+
         private
 
         def enough?(index:, direction:)
@@ -111,12 +127,12 @@ module Fusuma
         def interval_time(index:, direction:)
           @interval_time ||= {}
           @interval_time[index.cache_key] ||= begin
-                               keys_specific = Config::Index.new [*index.keys, 'interval']
-                               keys_global = Config::Index.new ['interval', direction]
-                               config_value = Config.search(keys_specific) ||
-                                              Config.search(keys_global) || 1
-                               BASE_INTERVAL * config_value
-                             end
+            keys_specific = Config::Index.new [*index.keys, 'interval']
+            keys_global = Config::Index.new ['interval', direction]
+            config_value = Config.search(keys_specific) ||
+                           Config.search(keys_global) || 1
+            BASE_INTERVAL * config_value
+          end
         end
       end
     end

data/lib/fusuma/plugin/parsers/tap_parser.rb

@@ -67,7 +67,11 @@ module Fusuma
 
             finger = case Regexp.last_match(2)
                      when 'TAP_STATE_DEAD'
-                       4
+                       if Regexp.last_match(1) == 'TAP_STATE_TOUCH_3'
+                         4
+                       else
+                         0
+                       end
                      when 'TAP_STATE_TOUCH_3'
                        3
                      when 'TAP_STATE_TOUCH_2'
@@ -94,8 +98,7 @@ module Fusuma
                        1
                      end
           # KEEP
-          when 'LIBINPUT TIMEOUT',   # Timeout Event
-            /\sgesture(| state):\s/  # 1.10.4 prints "gesture state: GESTURE_STATE_.*"
+          when /\sgesture(| state):\s/ # 1.10.4 prints "gesture state: GESTURE_STATE_.*"
             # NOTE: treat the "gesture(| state):" as KEEP
             status = 'keep'
             finger = 0
@@ -144,7 +147,7 @@ module Fusuma
           Events::Records::GestureRecord.new(status: status,
                                              gesture: gesture,
                                              finger: finger,
-                                             direction: nil)
+                                             delta: nil)
         end
 
         def tag

data/lib/fusuma/plugin/tap/version.rb

@@ -3,7 +3,7 @@
 module Fusuma
   module Plugin
     module Tap
-      VERSION = '0.3.0'
+      VERSION = '0.4.0'
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fusuma-plugin-tap
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - iberianpig
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2020-05-10 00:00:00.000000000 Z
+date: 2021-04-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: fusuma
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.10'
+        version: 2.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.10'
+        version: 2.0.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -169,7 +169,7 @@ homepage: https://github.com/iberianpig/fusuma-plugin-tap
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -184,8 +184,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.6
-signing_key: 
+rubygems_version: 3.1.4
+signing_key:
 specification_version: 4
 summary: Tap gesture plugin for Fusuma
 test_files: []