factbase 0.15.2 → 0.15.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 85e37453d7e238f20d15017e452fc39e2068cf86bc1c0bc0c5689439bf732c81
-  data.tar.gz: dc91da500dd5e512e041de74ad677dbf98ce24a84cb1972bc910884ddb3c48f3
+  metadata.gz: 5d4b1fc60e21bcadc4052ddc0de95b321d9b543fc2cde19ab33036ca473405f4
+  data.tar.gz: 07cc23815e363aff2995f29d359f93aa7c0916bd6bf59176018a297ef8d66679
 SHA512:
-  metadata.gz: 323d7a269d425e503365cd694cdafb224ee9bab256992d740af40c4151245d729cd183aba297f6bc963984d8e3d743ffe6e6099e6fbf3b0514a7994876d00533
-  data.tar.gz: 2ede879be19ca0faf61506cdfb9672117f767d66528c8fa5aa52237180848a011202eef6c3982b102e1b56430ebf9777b94fdfaf9b76e96738b25d45d1a549a3
+  metadata.gz: 4cb26f18ddfd83e7e709b3bf007a6f16905ef3dd2a6f6444d0fd401df675ec88d50c152c162a5ab1c3ba16362f3dadc1dc2c21eec12a223e427c4c285496d32c
+  data.tar.gz: 81a5f3ad8ca670537bff7f2a5739bd886f4cfc19146dde2dcf51e43789517e97a9ee5a54fc6aea0be1bc6f562e4774cbb3013501bcaaf03f5e901b444c6ed96f

data/lib/factbase/term.rb

@@ -121,7 +121,7 @@ class Factbase::Term
   rescue NoMethodError => e
     raise "Probably the term '#{@op}' is not defined at #{self}: #{e.message}"
   rescue StandardError => e
-    raise "#{e.message} at #{self}"
+    raise "#{e.message.inspect} at #{self} at #{e.backtrace[0]}"
   end
 
   # Simplify it if possible.

data/lib/factbase/terms/ordering.rb

@@ -20,24 +20,35 @@ module Factbase::Ordering
   end
 
   def unique(fact, maps, fb)
-    @uniques = {} if @uniques.nil?
+    @seen = Set.new if @seen.nil?
     raise "Too few operands for 'unique' (at least 1 expected)" if @operands.empty?
-    dups = 0
-    @operands.each_with_index do |_, i|
-      @uniques[i] = [] if @uniques[i].nil?
-      vv = _values(i, fact, maps, fb)
-      return false if vv.nil?
-      vv = [vv] unless vv.respond_to?(:to_a)
-      exists = false
-      vv.each do |v|
-        if @uniques[i].include?(v)
-          exists = true
-        else
-          @uniques[i] << v
-        end
-      end
-      dups += 1 if exists
+    vv = (0..(@operands.size - 1)).map { |i| _values(i, fact, maps, fb) }
+    return false if vv.any?(nil)
+    pass = true
+    _cartesian(vv).each do |t|
+      pass = false if @seen.include?(t)
+      @seen << t
+    end
+    pass
+  end
+
+  private
+
+  # Multiplies arrays and returns a list of all possible combinations
+  # of their elements. If this array is provided:
+  #
+  #  [ [4, 3], [2, 88, 13] ]
+  #
+  # This will be the result:
+  #
+  # [ [4, 2], [4, 88], [4, 13], [3, 2], [3, 88], [3, 13]]
+  def _cartesian(vv)
+    ff = vv.first.zip
+    if vv.one?
+      ff
+    else
+      tail = _cartesian(vv[1..])
+      ff.map { |f| tail.map { |t| f + t } }
     end
-    dups < @operands.size
   end
 end

data/lib/factbase/version.rb

@@ -9,5 +9,5 @@
 # License:: MIT
 class Factbase
   # Current version of the gem (changed by .rultor.yml on every release)
-  VERSION = '0.15.2' unless const_defined?(:VERSION)
+  VERSION = '0.15.3' unless const_defined?(:VERSION)
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: factbase
 version: !ruby/object:Gem::Version
-  version: 0.15.2
+  version: 0.15.3
 platform: ruby
 authors:
 - Yegor Bugayenko