Merge pull request #223 from Yaffle/master

peterolson · web-flow · commit 466959a66cbd · 2021-11-10T13:47:23.000-06:00
somehow fix the code to pass the test "is within 10% of uniform distr…
diff --git a/BigInteger.js b/BigInteger.js
@@ -1178,10 +1178,10 @@ var bigInt = (function (undefined) {
         var digits = toBase(range, BASE).value;
         var result = [], restricted = true;
         for (var i = 0; i < digits.length; i++) {
-            var top = restricted ? digits[i] : BASE;
+            var top = restricted ? digits[i] + (i + 1 < digits.length ? digits[i + 1] / BASE : 0) : BASE;
             var digit = truncate(usedRNG() * top);
             result.push(digit);
-            if (digit < top) restricted = false;
+            if (digit < digits[i]) restricted = false;
         }
         return low.add(Integer.fromArray(result, BASE, false));
     }

Original file line number	Diff line number	Diff line change
`@@ -1178,10 +1178,10 @@ var bigInt = (function (undefined) {`
`1178`	`1178`	`var digits = toBase(range, BASE).value;`
`1179`	`1179`	`var result = [], restricted = true;`
`1180`	`1180`	`for (var i = 0; i < digits.length; i++) {`
`1181`		`- var top = restricted ? digits[i] : BASE;`
	`1181`	`+ var top = restricted ? digits[i] + (i + 1 < digits.length ? digits[i + 1] / BASE : 0) : BASE;`
`1182`	`1182`	`var digit = truncate(usedRNG() * top);`
`1183`	`1183`	`result.push(digit);`
`1184`		`- if (digit < top) restricted = false;`
	`1184`	`+ if (digit < digits[i]) restricted = false;`
`1185`	`1185`	`}`
`1186`	`1186`	`return low.add(Integer.fromArray(result, BASE, false));`
`1187`	`1187`	`}`