@woosh/meep-engine 2.37.10 → 2.37.13

package/engine/ecs/transform/Transform.js

@@ -117,8 +117,6 @@ export class Transform {
 
     updateMatrix() {
         compose_matrix4_array(this.matrix, this.position, this.rotation, this.scale);
-
-        this.clearFlag(TransformFlags.MatrixNeedsUpdate);
     }
 
     /**
@@ -158,8 +156,6 @@ export class Transform {
         } else {
             this.scale.copy(Vector3.one);
         }
-
-        this.setFlag(TransformFlags.MatrixNeedsUpdate);
     }
 
     toJSON() {
@@ -236,14 +232,6 @@ export class Transform {
      * @param {Transform} b
      */
     multiplyTransforms(a, b) {
-        if (a.getFlag(TransformFlags.MatrixNeedsUpdate)) {
-            a.updateMatrix();
-        }
-
-        if (b.getFlag(TransformFlags.MatrixNeedsUpdate)) {
-            b.updateMatrix();
-        }
-
         mat4.multiply(m4_0, a.matrix, b.matrix);
 
         this.fromMatrix4(m4_0);
@@ -278,8 +266,6 @@ export class Transform {
 
         decompose_matrix_4_array(m, this.position, this.rotation, this.scale);
 
-        this.clearFlag(TransformFlags.MatrixNeedsUpdate);
-
         this.writeFlag(TransformFlags.AutomaticChangeDetection, ad);
     }
 

package/engine/ecs/transform/TransformFlags.js

@@ -3,6 +3,9 @@
  * @enum {number}
  */
 export const TransformFlags = {
+    /**
+     * @deprecated
+     */
     MatrixNeedsUpdate: 1,
     AutomaticChangeDetection: 2
 };

package/engine/graphics/ecs/mesh-v2/ShadedGeometry.js

@@ -3,8 +3,6 @@ import { AABB3 } from "../../../../core/bvh2/aabb3/AABB3.js";
 import { DrawMode } from "./DrawMode.js";
 import { ShadedGeometryFlags } from "./ShadedGeometryFlags.js";
 import { aabb3_matrix4_project } from "../../../../core/geom/3d/aabb/aabb3_matrix4_project.js";
-import { allocate_transform_m4 } from "./allocate_transform_m4.js";
-import { TransformFlags } from "../../../ecs/transform/TransformFlags.js";
 
 /**
  * @readonly
@@ -230,14 +228,6 @@ export class ShadedGeometry {
     }
 
     updateTransform() {
-        const t = this.__c_transform;
-
-        if (t.getFlag(TransformFlags.MatrixNeedsUpdate)) {
-            t.updateMatrix();
-        }
-
-        //this.__transform.set(t.matrix);
-
         this.update_bounds();
 
         this.__bvh_tree.node_move_aabb(this.__bvh_leaf_id, this.__bvh_aabb);

package/engine/graphics/ecs/mesh-v2/ShadedGeometrySystem.js

@@ -22,7 +22,6 @@ import TaskState from "../../../../core/process/task/TaskState.js";
 import {
     bvh_query_user_data_overlaps_frustum
 } from "../../../../core/bvh2/bvh3/bvh_query_user_data_overlaps_frustum.js";
-import { TransformFlags } from "../../../ecs/transform/TransformFlags.js";
 
 
 /**
@@ -37,6 +36,10 @@ const scratch_point = new SurfacePoint3();
  */
 const scratch_m4 = new Float32Array(16);
 
+/**
+ * @readonly
+ * @type {Float32Array}
+ */
 const scratch_ray_0 = new Float32Array(6);
 
 export class ShadedGeometrySystem extends System {
@@ -251,23 +254,21 @@ export class ShadedGeometrySystem extends System {
         t.rotation.onChanged.add(sg.updateTransform, sg);
         t.scale.onChanged.add(sg.updateTransform, sg);
 
-        if (t.getFlag(TransformFlags.MatrixNeedsUpdate)) {
-            t.updateMatrix();
-        }
-
-        // sg.__transform.set(t.matrix);
         sg.update_bounds();
 
         // remember entity for lookups
         sg.__entity = entity;
 
         // insert BVH entry
-        const bvh_node_id = this.__bvh_binary.allocate_node();
-        this.__bvh_binary.node_set_aabb(bvh_node_id, sg.__bvh_aabb);
-        this.__bvh_binary.node_set_user_data(bvh_node_id, entity);
-        this.__bvh_binary.insert_leaf(bvh_node_id);
+        const bvh = this.__bvh_binary;
+
+        const bvh_node_id = bvh.allocate_node();
+
+        bvh.node_set_aabb(bvh_node_id, sg.__bvh_aabb);
+        bvh.node_set_user_data(bvh_node_id, entity);
+        bvh.insert_leaf(bvh_node_id);
 
-        sg.__bvh_tree = this.__bvh_binary;
+        sg.__bvh_tree = bvh;
         sg.__bvh_leaf_id = bvh_node_id;
 
 

package/engine/graphics/ecs/mesh-v2/aggregate/SGMesh.d.ts

@@ -8,5 +8,7 @@ export class SGMesh {
 
     getBoundingBox(out: AABB3): boolean
 
+    getUntransformedBoundingBox(out: AABB3): void
+
     static fromURL(url: string): SGMesh
 }

package/engine/graphics/ecs/mesh-v2/aggregate/SGMesh.js

@@ -1,4 +1,5 @@
 import { entity_node_compute_bounding_box } from "../../../../ecs/parent/entity_node_compute_bounding_box.js";
+import { AABB3 } from "../../../../../core/bvh2/aabb3/AABB3.js";
 
 export const SGMeshFlags = {
     CastShadow: 1,
@@ -20,6 +21,13 @@ export class SGMesh {
          */
         this.__node = null;
 
+        /**
+         *
+         * @type {AABB3}
+         * @private
+         */
+        this.__initial_bounds = new AABB3();
+
         /**
          *
          * @type {number}
@@ -63,6 +71,14 @@ export class SGMesh {
         return true;
     }
 
+    /**
+     *
+     * @param {AABB3} destination
+     */
+    getUntransformedBoundingBox(destination) {
+        destination.copy(this.__initial_bounds);
+    }
+
     /**
      *
      * @param {number|SGMeshFlags} flag

package/engine/graphics/ecs/mesh-v2/aggregate/SGMeshSystem.js

@@ -76,7 +76,21 @@ export class SGMeshSystem extends System {
             return;
         }
 
-        // TODO check that entity still exists, as we get to this point asynchronously
+        if (!ecd.entityExists(entity)) {
+            // entity no longer exists
+            return;
+        }
+
+        if (ecd.getComponent(entity, SGMesh) !== mesh) {
+            // mesh has changed
+            return;
+        }
+
+        if (asset.boundingBox !== undefined) {
+            mesh.__initial_bounds.copy(asset.boundingBox);
+        } else {
+            console.error(`No bounds for asset: ${asset.description}`);
+        }
 
         let entity_node = three_object_to_entity_composition(asset.create());
 
@@ -145,6 +159,14 @@ export class SGMeshSystem extends System {
      * @param {number} entity
      */
     link(mesh, transform, entity) {
+        const p = transform.position;
+
+        // set initial bounds to single point
+        mesh.__initial_bounds.setBounds(
+            p.x, p.y, p.z,
+            p.x, p.y, p.z,
+        );
+
         if (mesh.url === null) {
             this.__wait_queue.push(entity);
             return;

package/engine/graphics/ecs/mesh-v2/render/adapters/GenericRendererAdapter.js

@@ -1,5 +1,4 @@
 import { AbstractRenderAdapter } from "./AbstractRenderAdapter.js";
-import { array_copy } from "../../../../../../core/collection/array/copyArray.js";
 import { SGThreeObjectCache } from "../SGThreeObjectCache.js";
 
 export class GenericRendererAdapter extends AbstractRenderAdapter {
@@ -30,7 +29,13 @@ export class GenericRendererAdapter extends AbstractRenderAdapter {
          */
         const object = this.__cache.get(sg);
 
-        array_copy(sg.transform, 0, object.matrixWorld.elements, 0, 16);
+        const source = sg.transform;
+        const destination = object.matrixWorld.elements;
+
+        // copy transform matrix
+        for (let i = 0; i < 16; i++) {
+            destination[i] = source[i];
+        }
 
         this.__objects[this.__object_count++] = object;
     }

package/package.json

@@ -5,7 +5,7 @@
   "productName": "Meep",
   "description": "production-ready JavaScript game engine based on Entity Component System Architecture",
   "author": "Alexander Goldring",
-  "version": "2.37.10",
+  "version": "2.37.13",
   "dependencies": {
     "gl-matrix": "3.4.3",
     "fast-levenshtein": "2.0.6",