vose 0.2.0 → 0.2.1

data/lib/vose.rb

@@ -4,7 +4,7 @@ module Vose
   class InvalidArgumentException < RuntimeError; end;
 
   class AliasMethod
-    attr_reader :limit, :prob
+    attr_reader :limit
 
     def initialize(probabilities)
       raise InvalidArgumentException if probabilities.empty?
@@ -33,24 +33,27 @@ module Vose
 
       0.upto(limit-1) do |j|
         if scaled_probality[j] > 1
-          large_worklist[large_worklist_counter+=1] = j
+          large_worklist[large_worklist_counter] = j
+          large_worklist_counter+=1
         else
-          small_worklist[small_worklist_counter+=1] = j
+          small_worklist[small_worklist_counter] = j
+          small_worklist_counter+=1
         end
       end
 
       while small_worklist_counter != 0 && large_worklist_counter != 0
-        current_small_worklist_index = small_worklist[small_worklist_counter].to_i
-        current_large_worklist_index = large_worklist[large_worklist_counter].to_i
         small_worklist_counter-=1
         large_worklist_counter-=1
 
+        current_small_worklist_index = small_worklist[small_worklist_counter].to_i
+        current_large_worklist_index = large_worklist[large_worklist_counter].to_i
+
         @prob[current_small_worklist_index] = scaled_probality[current_small_worklist_index]
         @alias[current_small_worklist_index] = current_large_worklist_index
 
         scaled_probality[current_large_worklist_index] = (scaled_probality[current_large_worklist_index] + scaled_probality[current_small_worklist_index]) - 1
         if scaled_probality[current_large_worklist_index] > 1
-          large_worklist[large_worklist_counter] = large_worklist_counter
+          large_worklist[large_worklist_counter] = current_large_worklist_index
           large_worklist_counter+=1
         else
           small_worklist[small_worklist_counter] = current_large_worklist_index

data/lib/vose/version.rb

@@ -1,3 +1,3 @@
 module Vose
-  VERSION = "0.2.0"
+  VERSION = "0.2.1"
 end

data/spec/vose_spec.rb

@@ -8,14 +8,14 @@ describe Vose::AliasMethod do
     }.must_raise(Vose::InvalidArgumentException)
   end
 
-  it "complinas if the probabilities aren't positive" do
+  it "complains if the probabilities aren't positive" do
     probabilities = [-0.5]
     lambda { 
       Vose::AliasMethod.new probabilities
     }.must_raise(Vose::InvalidArgumentException)
   end  
 
-  it "sets the limit to the lenght of probabilities" do
+  it "sets the limit to the length of probabilities" do
     probabilities = [0.1, 0.9]
     vose = Vose::AliasMethod.new probabilities
     vose.limit.must_equal 2
@@ -36,7 +36,7 @@ describe Vose::AliasMethod do
       results.must_match_probability expected_percentages
     end
 
-    it "allows you choose the next probablity given your inputs" do
+    it "allows you choose the next probability given your inputs" do
       probabilities = [0.1, 0.5, 0.4]
       vose = Vose::AliasMethod.new probabilities
       results = []

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: vose
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-07-22 00:00:00.000000000 Z
+date: 2012-07-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pry