@apache-arrow/es2015-cjs - npm Package Compare versions

Comparing version 0.15.1 to 0.16.0

compute/dataframe.js

@@ -129,12 +129,13 @@ "use strict";
             const batch = batches[batchIndex];
-            // TODO: bind batches lazily
-            // If predicate doesn't match anything in the batch we don't need
-            // to bind the callback
-            if (bind) {
-                bind(batch);
-            }
             const predicate = this._predicate.bind(batch);
+            let isBound = false;
             // yield all indices
             for (let index = -1, numRows = batch.length; ++index < numRows;) {
                 if (predicate(index, batch)) {
+                    // bind batches lazily - if predicate doesn't match anything
+                    // in the batch we don't need to call bind on the batch
+                    if (bind && !isBound) {
+                        bind(batch);
+                        isBound = true;
+                    }
                     next(index, batch);
@@ -151,12 +152,13 @@ }
             const batch = batches[batchIndex];
-            // TODO: bind batches lazily
-            // If predicate doesn't match anything in the batch we don't need
-            // to bind the callback
-            if (bind) {
-                bind(batch);
-            }
             const predicate = this._predicate.bind(batch);
+            let isBound = false;
             // yield all indices
             for (let index = batch.length; --index >= 0;) {
                 if (predicate(index, batch)) {
+                    // bind batches lazily - if predicate doesn't match anything
+                    // in the batch we don't need to call bind on the batch
+                    if (bind && !isBound) {
+                        bind(batch);
+                        isBound = true;
+                    }
                     next(index, batch);
@@ -163,0 +165,0 @@ }

compute/predicate.js

@@ -128,3 +128,3 @@ "use strict";
 exports.CombinationPredicate = CombinationPredicate;
-// add children to protoype so it doesn't get mangled in es2015/umd
+// add children to prototype so it doesn't get mangled in es2015/umd
 CombinationPredicate.prototype.children = Object.freeze([]); // freeze for safety
@@ -204,3 +204,3 @@ /** @ignore */
     _bindLitCol(batch, lit, col) {
-        // Equals is comutative
+        // Equals is commutative
         return this._bindColLit(batch, col, lit);
@@ -207,0 +207,0 @@ }

package.json

 {
-  "version": "0.15.1",
+  "version": "0.16.0",
   "name": "@apache-arrow/es2015-cjs",
@@ -4,0 +4,0 @@ "browser": "Arrow.dom.js",

README.md

@@ -46,2 +46,3 @@ <!---
 * [Observable: Manipulating flat arrays arrow-style][6]
+* [Observable: Rich columnar data tables - Dictionary-encoded strings, 64bit ints, and nested structs][8]
 * [/js/test/unit](https://github.com/apache/arrow/tree/master/js/test/unit) - Unit tests for Table and Vector
@@ -281,1 +282,2 @@
 [7]: https://arrow.apache.org/docs/js/
+[8]: https://observablehq.com/@lmeyerov/rich-data-types-in-apache-arrow-js-efficient-data-tables-wit

type.js

@@ -379,3 +379,2 @@ "use strict";
         this.children = children;
-        this.children = children;
     }
@@ -382,0 +381,0 @@ get typeId() { return enum_1.Type.Struct; }

util/bn.d.ts

@@ -42,3 +42,3 @@ import { ArrayBufferViewInput } from './buffer';
      * Down-convert the bytes to a 53-bit precision integer. Invoked by JS for
-     * arithmatic operators, like `+`. Easy (and unsafe) way to convert BN to
+     * arithmetic operators, like `+`. Easy (and unsafe) way to convert BN to
      * number via `+bn_inst`
@@ -45,0 +45,0 @@ */

util/vector.js

@@ -92,7 +92,7 @@ "use strict";
 function createArrayLikeComparator(lhs) {
-    const comparitors = [];
+    const comparators = [];
     for (let i = -1, n = lhs.length; ++i < n;) {
-        comparitors[i] = createElementComparator(lhs[i]);
+        comparators[i] = createElementComparator(lhs[i]);
     }
-    return createSubElementsComparator(comparitors);
+    return createSubElementsComparator(comparators);
 }
@@ -102,13 +102,13 @@ /** @ignore */
     let i = -1;
-    const comparitors = [];
-    lhs.forEach((v) => comparitors[++i] = createElementComparator(v));
-    return createSubElementsComparator(comparitors);
+    const comparators = [];
+    lhs.forEach((v) => comparators[++i] = createElementComparator(v));
+    return createSubElementsComparator(comparators);
 }
 /** @ignore */
 function createVectorComparator(lhs) {
-    const comparitors = [];
+    const comparators = [];
     for (let i = -1, n = lhs.length; ++i < n;) {
-        comparitors[i] = createElementComparator(lhs.get(i));
+        comparators[i] = createElementComparator(lhs.get(i));
     }
-    return createSubElementsComparator(comparitors);
+    return createSubElementsComparator(comparators);
 }
@@ -122,9 +122,9 @@ /** @ignore */
     }
-    const comparitors = [];
+    const comparators = [];
     for (let i = -1, n = keys.length; ++i < n;) {
-        comparitors[i] = createElementComparator(lhs[keys[i]]);
+        comparators[i] = createElementComparator(lhs[keys[i]]);
     }
-    return createSubElementsComparator(comparitors, keys);
+    return createSubElementsComparator(comparators, keys);
 }
-function createSubElementsComparator(comparitors, keys) {
+function createSubElementsComparator(comparators, keys) {
     return (rhs) => {
@@ -135,16 +135,16 @@ if (!rhs || typeof rhs !== 'object') {
         switch (rhs.constructor) {
-            case Array: return compareArray(comparitors, rhs);
+            case Array: return compareArray(comparators, rhs);
             case Map:
             case row_1.MapRow:
             case row_1.StructRow:
-                return compareObject(comparitors, rhs, rhs.keys());
+                return compareObject(comparators, rhs, rhs.keys());
             case Object:
             case undefined: // support `Object.create(null)` objects
-                return compareObject(comparitors, rhs, keys || Object.keys(rhs));
+                return compareObject(comparators, rhs, keys || Object.keys(rhs));
         }
-        return rhs instanceof vector_1.Vector ? compareVector(comparitors, rhs) : false;
+        return rhs instanceof vector_1.Vector ? compareVector(comparators, rhs) : false;
     };
 }
-function compareArray(comparitors, arr) {
-    const n = comparitors.length;
+function compareArray(comparators, arr) {
+    const n = comparators.length;
     if (arr.length !== n) {
@@ -154,3 +154,3 @@ return false;
     for (let i = -1; ++i < n;) {
-        if (!(comparitors[i](arr[i]))) {
+        if (!(comparators[i](arr[i]))) {
             return false;
@@ -161,4 +161,4 @@ }
 }
-function compareVector(comparitors, vec) {
-    const n = comparitors.length;
+function compareVector(comparators, vec) {
+    const n = comparators.length;
     if (vec.length !== n) {
@@ -168,3 +168,3 @@ return false;
     for (let i = -1; ++i < n;) {
-        if (!(comparitors[i](vec.get(i)))) {
+        if (!(comparators[i](vec.get(i)))) {
             return false;
@@ -175,3 +175,3 @@ }
 }
-function compareObject(comparitors, obj, keys) {
+function compareObject(comparators, obj, keys) {
     const lKeyItr = keys[Symbol.iterator]();
@@ -181,3 +181,3 @@ const rKeyItr = obj instanceof Map ? obj.keys() : Object.keys(obj)[Symbol.iterator]();
     let i = 0;
-    let n = comparitors.length;
+    let n = comparators.length;
     let rVal = rValItr.next();
@@ -187,3 +187,3 @@ let lKey = lKeyItr.next();
     for (; i < n && !lKey.done && !rKey.done && !rVal.done; ++i, lKey = lKeyItr.next(), rKey = rKeyItr.next(), rVal = rValItr.next()) {
-        if (lKey.value !== rKey.value || !comparitors[i](rVal.value)) {
+        if (lKey.value !== rKey.value || !comparators[i](rVal.value)) {
             break;
@@ -190,0 +190,0 @@ }

New alerts

Mixed license

License

(Experimental) Package contains multiple licenses.

Found 1 instance in 1 package

Unidentified License

License

(Experimental) Something that seems like a license was found, but its contents could not be matched with a known license.

Found 1 instance in 1 package

Fixed alerts

Mixed license

License

(Experimental) Package contains multiple licenses.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

2579692

0.14%

Lines of code

24215

0

Number of lines in readme file

282

0.71%

Worsened metrics

Number of low license alerts

2

100%

No dependency changes