@woosh/meep-engine 2.84.5 → 2.84.6

package/build/meep.module.js

@@ -61269,7 +61269,7 @@ class List {
         this.data.push(el);
         const oldLength = this.length;
 
-        this.length++;
+        this.length = oldLength + 1;
 
         this.on.added.send2(el, oldLength);
         return this;
@@ -96084,40 +96084,38 @@ LinearValue.prototype.fromJSON = function (json) {
 
 /**
  *
- * @param {number} x
- * @param {string} [separator=',']
+ * @param {string} string
+ * @param {string} trailing_sequence
  * @returns {string}
  */
-function number_format_by_thousands(x, separator = ',') {
+function string_strip_trailing(string, trailing_sequence) {
+    const trailing_sequence_length = trailing_sequence.length;
 
-    return x.toString().replace(/\B(?=(\d{3})+(?!\d))/g, separator);
+    if (trailing_sequence_length <= 0) {
+        // special case to avoid infinite looping
+        return string;
+    }
+
+    let end_index = string.length;
+
+    while (string.substring(end_index - trailing_sequence_length, end_index) === trailing_sequence) {
+        end_index -= trailing_sequence_length;
+    }
+
+    return string.substring(0, end_index);
 }
 
 /**
  *
- * @param {string} value
- * @returns {number}
+ * @param {number} x
+ * @param {string} [separator=',']
+ * @returns {string}
  */
-function countDecimals(value) {
-    if (value % 1 === 0) {
-        //whole number
-        return 0;
-    }
-    const s = value.toString();
-    const index = s.indexOf('.');
-
-    if (index === -1) {
-        return 0;
-    }
-
-    //find last 0
-    let endIndex = s.length - 1;
-    for (; endIndex > index && s.charAt(endIndex) === "0"; endIndex--) {
-
-    }
-    return endIndex - index;
-}
+function number_format_by_thousands(x, separator = ',') {
 
+    return x.toString().replace(/\B(?=(\d{3})+(?!\d))/g, separator);
+}
+
 /**
  *
  * @param {number} value
@@ -96128,10 +96126,12 @@ function number_pretty_print(value) {
     const MAX_DECIMALS = 2;
 
     const fraction = value % 1;
-    if (fraction !== 0 && Math.abs(value) < 100) {
-        const decimals = countDecimals(value.toFixed(MAX_DECIMALS));
-        const decimalsToPrint = Math.min(decimals, MAX_DECIMALS);
-        return value.toFixed(decimalsToPrint);
+
+    const would_produce_decimals = fraction * Math.pow(10, MAX_DECIMALS) > 0;
+
+    if (would_produce_decimals && Math.abs(value) < 100) {
+        const truncated = value.toFixed(MAX_DECIMALS);
+        return string_strip_trailing(truncated, "0");
     } else {
         //no fraction
         return number_format_by_thousands(value - fraction, ",");

package/package.json

@@ -5,7 +5,7 @@
   "description": "Fully featured ECS game engine written in JavaScript",
   "type": "module",
   "author": "Alexander Goldring",
-  "version": "2.84.5",
+  "version": "2.84.6",
   "main": "build/meep.module.js",
   "module": "build/meep.module.js",
   "exports": {

package/src/core/collection/list/List.js

@@ -99,7 +99,7 @@ class List {
         this.data.push(el);
         const oldLength = this.length;
 
-        this.length++;
+        this.length = oldLength + 1;
 
         this.on.added.send2(el, oldLength);
         return this;

package/src/core/primitives/numbers/number_count_decimals.js

@@ -0,0 +1,30 @@
+import { assert } from "../../assert.js";
+
+/**
+ *
+ * @param {number} value
+ * @returns {number}
+ */
+export function number_count_decimals(value) {
+    assert.isNumber(value, 'value');
+
+    if (value % 1 === 0) {
+        //whole number
+        return 0;
+    }
+
+    const s = value.toString();
+    const index = s.indexOf('.');
+
+    if (index === -1) {
+        return 0;
+    }
+
+    //find last 0
+    let endIndex = s.length - 1;
+    for (; endIndex > index && s.charAt(endIndex) === "0"; endIndex--) {
+
+    }
+
+    return endIndex - index;
+}

package/src/core/primitives/numbers/number_pretty_print.js

@@ -1,31 +1,7 @@
 import { assert } from "../../assert.js";
+import { string_strip_trailing } from "../strings/string_strip_trailing.js";
 import { number_format_by_thousands } from "./number_format_by_thousands.js";
 
-/**
- *
- * @param {string} value
- * @returns {number}
- */
-function countDecimals(value) {
-    if (value % 1 === 0) {
-        //whole number
-        return 0;
-    }
-    const s = value.toString();
-    const index = s.indexOf('.');
-
-    if (index === -1) {
-        return 0;
-    }
-
-    //find last 0
-    let endIndex = s.length - 1;
-    for (; endIndex > index && s.charAt(endIndex) === "0"; endIndex--) {
-
-    }
-    return endIndex - index;
-}
-
 /**
  *
  * @param {number} value
@@ -37,10 +13,12 @@ export function number_pretty_print(value) {
     const MAX_DECIMALS = 2;
 
     const fraction = value % 1;
-    if (fraction !== 0 && Math.abs(value) < 100) {
-        const decimals = countDecimals(value.toFixed(MAX_DECIMALS));
-        const decimalsToPrint = Math.min(decimals, MAX_DECIMALS);
-        return value.toFixed(decimalsToPrint);
+
+    const would_produce_decimals = fraction * Math.pow(10, MAX_DECIMALS) > 0;
+
+    if (would_produce_decimals && Math.abs(value) < 100) {
+        const truncated = value.toFixed(MAX_DECIMALS);
+        return string_strip_trailing(truncated, "0");
     } else {
         //no fraction
         return number_format_by_thousands(value - fraction, ",");

package/src/core/primitives/strings/string_format_camel_to_kebab.js

@@ -7,3 +7,5 @@
 export function string_format_camel_to_kebab(string) {
     return string.replace(/([a-z])([A-Z])/g, '$1-$2').toLowerCase();
 }
+
+

package/src/core/primitives/strings/string_strip_trailing.js

@@ -0,0 +1,22 @@
+/**
+ *
+ * @param {string} string
+ * @param {string} trailing_sequence
+ * @returns {string}
+ */
+export function string_strip_trailing(string, trailing_sequence) {
+    const trailing_sequence_length = trailing_sequence.length;
+
+    if (trailing_sequence_length <= 0) {
+        // special case to avoid infinite looping
+        return string;
+    }
+
+    let end_index = string.length;
+
+    while (string.substring(end_index - trailing_sequence_length, end_index) === trailing_sequence) {
+        end_index -= trailing_sequence_length;
+    }
+
+    return string.substring(0, end_index);
+}

package/src/core/primitives/strings/string_strip_trailing.spec.js

@@ -0,0 +1,27 @@
+import { string_strip_trailing } from "./string_strip_trailing.js";
+
+test("strip empty from empty", () => {
+    expect(
+        string_strip_trailing("", "")
+    ).toBe("");
+});
+
+test("strip repeating 2 character sequence", () => {
+
+    expect(
+        string_strip_trailing("Cabab", "ab")
+    ).toBe("C");
+
+});
+
+test("mistakes", () => {
+
+    expect(
+        string_strip_trailing("food", "o")
+    ).toBe("food");
+
+    expect(
+        string_strip_trailing("food", "f")
+    ).toBe("food");
+
+});

package/src/engine/ecs/storage/BinaryBufferDeSerializer.js

@@ -1,4 +1,5 @@
 import { assert } from "../../../core/assert.js";
+import { number_pretty_print } from "../../../core/primitives/numbers/number_pretty_print.js";
 import Task from "../../../core/process/task/Task.js";
 import { TaskSignal } from "../../../core/process/task/TaskSignal.js";
 import { emptyTask } from "../../../core/process/task/util/emptyTask.js";
@@ -24,6 +25,7 @@ class BinaryBufferDeSerializer {
         assert.notEqual(engine, undefined, 'engine is undefined');
         assert.notEqual(dataset, undefined, 'dataset is undefined');
 
+        const entity_count_before = dataset.entityCount;
 
         const version = buffer.readUint16();
         const numSerializedTypes = buffer.readUint16();
@@ -37,7 +39,12 @@ class BinaryBufferDeSerializer {
         }
 
         task.on.completed.add(function () {
-            console.log(`Binary Buffer De-Serialization took ${task.getExecutedCpuTime()}ms`);
+            const entity_count_after = dataset.entityCount;
+
+            const execution_time = task.getExecutedCpuTime();
+            const entity_count = entity_count_after - entity_count_before;
+
+            console.log(`Binary Buffer De-Serialization took ${number_pretty_print(execution_time)}ms. ${entity_count} Entities`);
         });
 
         return task;