opener-property-tagger 3.3.6 → 3.4.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0bdc478005838e629b52e310437c5cac89158b8f33429d477c7096f1210c3249
-  data.tar.gz: a8fec4e6229cfd39f1cc0f11c3a8007c2c40442fd6ed2f00a558d2cbd8f01c8d
+  metadata.gz: 2d223b859da75d45b68da9def103878187f20a8f95c780eaf73556d5b4d3eb5e
+  data.tar.gz: 021f008d00c3cbcf640f694703ca4308f77d080e0ab85ff72c89570a9f6987e7
 SHA512:
-  metadata.gz: a88d97bd1fff6f57c956024d7d47affe13179a02e068129ddda1be0c577324eafee18c271e465d68dc06a99d7cdd78288e6094f307ab4503ee68c963e618bafc
-  data.tar.gz: 6e601c83e09931821e6d00c03114fbaabb72f8c72781f667eea384b7beb916adc3ccc42f83dbd15434779e2dd3e5f574bcb8ef7be022f3154f0afdabec0107b2
+  metadata.gz: fce119a3e41bc3647816c45fb328d3c626ebc7d8bbfe2d2372931d612da3aa7db1e8cd6591c02c6d512532736e5b820ae707fa7616114028165e37ceb073e495
+  data.tar.gz: 307786c8aa77e7785486d1e8eebc506dd7096861702855d2f2f0d2fe12b216be3764900de415e492f9d236536b41204e6cfe566cae8a92fa0b71b39363cf41f6

data/lib/opener/property_tagger/file_aspects_cache.rb

@@ -33,12 +33,17 @@ module Opener
       # @param [String] path
       #
       def load_aspects(path)
-        mapping = Hash.new { |hash, key| hash[key] = [] }
+        mapping = Hash.new{ |hash, key| hash[key] = [] }
 
-        File.foreach(path) do |line|
-          lemma, _pos, aspect = line.chomp.split("\t")
+        File.foreach path do |line|
+          lemma, pos, aspect = line.chomp.split("\t")
+          l = Hashie::Mash.new(
+            lemma:  lemma,
+            pos:    pos,
+            aspect: aspect,
+          )
 
-          mapping[lemma.to_sym] << aspect
+          mapping[l.lemma.to_sym] << l
         end
 
         return mapping

data/lib/opener/property_tagger/processor.rb

@@ -6,7 +6,8 @@ module Opener
     class Processor
 
       attr_accessor :document
-      attr_accessor :aspects, :aspects_path, :aspects_url
+      attr_accessor :aspects_path, :aspects_url
+      attr_accessor :aspects, :lexicons
       attr_accessor :timestamp, :pretty
 
       ##
@@ -34,10 +35,10 @@ module Opener
         @remote       = !url.nil?
         @aspects_path = path
         @aspects_url  = url
-        @cache_keys   = params[:cache_keys]
+        @cache_keys   = params[:cache_keys] || {}
         @cache_keys.merge! lang: @document.root.attr('xml:lang')
 
-        @aspects = if @remote then REMOTE_ASPECTS_CACHE[**@cache_keys].aspects else FILE_ASPECTS_CACHE[aspects_file] end
+        @lexicons = if @remote then REMOTE_ASPECTS_CACHE[**@cache_keys].aspects else FILE_ASPECTS_CACHE[aspects_file] end
       end
 
       ##
@@ -45,15 +46,13 @@ module Opener
       # @return [String]
       #
       def process
-        existing_aspects = extract_aspects
-
         add_features_layer
         add_properties_layer
 
-        existing_aspects.each_with_index do |(key, value), index|
+        extract_aspects.each.with_index do |(lemma, values), index|
           index += 1
 
-          add_property(key, value, index)
+          add_property lemma, values, index
         end
 
         add_linguistic_processor
@@ -77,37 +76,41 @@ module Opener
         @terms
       end
 
+      # Use of n-grams to determine if a unigram (1 lemma) or bigram (2
+      # lemmas) belong to a property.
+      MAX_NGRAM = 2
+
       ##
       # Check which terms belong to an aspect (property)
       # Text have priority over Lemmas, overriding if there is a conflict
       # @return [Hash]
       #
       def extract_aspects
-        term_ids     = terms.keys
+        all_term_ids = terms.keys
         lemmas       = terms.values
-        uniq_aspects = Hash.new { |hash, key| hash[key] = [] }
+        uniq_aspects = Hash.new{ |hash, lemma| hash[lemma] = [] }
 
         [:lemma, :text].each do |k|
           current_token = 0
-          # Use of n-grams to determine if a unigram (1 lemma) or bigram (2
-          # lemmas) belong to a property.
-          max_ngram = 2
-
 
           while current_token < terms.count
-            (0..max_ngram).each do |tam_ngram|
-              if current_token + tam_ngram <= terms.count
-                ngram = lemmas[current_token..current_token+tam_ngram].map{|a| a[k] }.join(" ").downcase
-
-                if aspects[ngram.to_sym]
-                  properties = aspects[ngram.to_sym]
-                  ids        = term_ids[current_token..current_token+tam_ngram]
-
-                  properties.uniq.each do |property|
-                    next if !property or property.strip.empty?
-
-                    uniq_aspects[property.to_sym] << [ids,ngram] unless uniq_aspects[property.to_sym].include? [ids,ngram]
-                  end
+            (0..MAX_NGRAM).each do |tam_ngram|
+              next unless current_token + tam_ngram <= terms.count
+
+              ngram = lemmas[current_token..current_token+tam_ngram].map{ |a| a[k] }.join(" ").downcase
+
+              @lexicons[ngram.to_sym]&.each do |l|
+                properties = if l.aspects.present? then l.aspects else [l.aspect] end
+                properties.each do |p|
+                  next if p.blank?
+                  term_ids = all_term_ids[current_token..current_token+tam_ngram]
+                  next if uniq_aspects[p.to_sym].find{ |v| v.term_ids == term_ids }
+
+                  uniq_aspects[p.to_sym] << Hashie::Mash.new(
+                    term_ids: term_ids,
+                    ngram:    ngram,
+                    lexicon:  l,
+                  )
                 end
               end
             end
@@ -135,24 +138,25 @@ module Opener
         new_node("properties", "KAF/features")
       end
 
-      def add_property(key, value, index)
+      def add_property lemma, values, index
         property_node = new_node("property", "KAF/features/properties")
 
-        property_node['lemma'] = key.to_s
+        property_node['lemma'] = lemma.to_s
         property_node['pid']   = "p#{index.to_s}"
 
         references_node = new_node("references", property_node)
 
-        value.uniq.each do |v|
-          comm_node = Nokogiri::XML::Comment.new(references_node, " #{v.last} ")
+        values.each do |v|
+          comm_node = Nokogiri::XML::Comment.new(references_node, " #{v.ngram} ")
           references_node.add_child comm_node
 
-          span_node = new_node("span", references_node)
+          span_node = new_node 'span', references_node
 
-          v.first.each do |val|
-            target_node       = new_node("target", span_node)
+          v.term_ids.each do |id|
+            target_node       = new_node 'target', span_node
 
-            target_node['id'] = val.to_s
+            target_node['id'] = id.to_s
+            target_node['lexicon-id'] = v.lexicon.id if v.lexicon.id
           end
         end
       end

data/lib/opener/property_tagger/remote_aspects_cache.rb

@@ -17,10 +17,11 @@ module Opener
       end
 
       def [] **params
+        existing = @cache[params]
+        return existing if existing and existing.from > UPDATE_INTERVAL.ago
+
         synchronize do
-          existing = @cache[params]
-          break existing if existing and existing.from > UPDATE_INTERVAL.ago
-          @cache[params] = cache_update existing, **params
+          @cache[params] = cache_update @cache[params], **params
         end
       end
       alias_method :get, :[]
@@ -49,7 +50,10 @@ module Opener
         lexicons = lexicons['data'].map{ |l| Hashie::Mash.new l }
         mapping  = Hash.new{ |hash, key| hash[key] = [] }
         lexicons.each do |l|
-          mapping[l.lemma.to_sym] << l.aspect
+          mapping[l.lemma.to_sym] << l
+          l.variants&.each do |v|
+            mapping[v.lemma.to_sym] << l
+          end
         end
 
         mapping

data/lib/opener/property_tagger/version.rb

@@ -1,7 +1,7 @@
 module Opener
   class PropertyTagger
 
-    VERSION = '3.3.6'
+    VERSION = '3.4.3'
 
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: opener-property-tagger
 version: !ruby/object:Gem::Version
-  version: 3.3.6
+  version: 3.4.3
 platform: ruby
 authors:
 - development@olery.com
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-12-10 00:00:00.000000000 Z
+date: 2021-09-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: opener-daemons
@@ -167,9 +167,9 @@ dependencies:
 description: Property tagger for hotels in Dutch and English.
 email:
 executables:
-- property-tagger
 - property-tagger-daemon
 - property-tagger-server
+- property-tagger
 extensions: []
 extra_rdoc_files: []
 files:
@@ -212,8 +212,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project:
-rubygems_version: 2.7.8
+rubygems_version: 3.2.14
 signing_key:
 specification_version: 4
 summary: Property tagger for hotels in Dutch and English.