Automatic recalculation of scene-bbox on scene update and near/far clip optimization for perspective camera.

* GlViewerRCore.js:
- Add this.scene_bbox, add flag recalc_bbox to request_render()
- Call camera.optimizNearFar(scene_bbox) for perspective camera
- ApplyTemporaryRCoreExtensions() -- that is not used at this point

* EveScene.js
- Add comment about better handling of scene / viewer bounding boxes.
- In Scene::endChanges() call render via request_render and request bbox recalc.

Author: osschar

ui5/eve7/lib/EveScene.js

@@ -201,8 +201,15 @@ sap.ui.define(['rootui5/eve7/lib/EveManager'], function(EveManager) {
                this.update3DObjectsVisibility(p.childs, true);
                this.need_visibility_update = false;
             }
-            this.glctrl.viewer.render();
 
+            // To improve when bbox info is streamed.
+            // Recalc scene bbox -- and update viewer total bbox, from scene boxes.
+            // 1. recalc-scene-bbox from known element bboxes (streamed)
+            //    [ this could really be done on the server ]
+            // 2. tell viewer to recalc-total-bbox ONLY from scene bboxes.
+
+            // For now just refresh the gl-viewer.
+            this.glctrl.viewer.request_render(true);
          }
       }
 

ui5/eve7/lib/GlViewerRCore.js

@@ -87,6 +87,8 @@ sap.ui.define([
                   return canvasDOM;
                };
 
+               pthis.ApplyTemporaryRCoreExtensions();
+
                pthis.bootstrap();
             });
          } else {
@@ -203,6 +205,7 @@ sap.ui.define([
          this.renderer.clearColor = "#00000000";
          this.scene = new RC.Scene();
          this.overlay_scene = new RC.Scene();
+         this.scene_bbox = new RC.Box3();
 
          this.lights = new RC.Group;
          this.lights.name = "Light container";
@@ -470,18 +473,23 @@ sap.ui.define([
          this.resetRenderer();
       }
 
-      resetRenderer()
+      recalcSceneBBox()
       {
-         let sbbox = new RC.Box3();
-         sbbox.setFromObject( this.scene );
-         if (sbbox.isEmpty())
+         this.scene_bbox.setFromObject( this.scene );
+         if (this.scene_bbox.isEmpty())
          {
-            console.error("GlViewerRenderCore.resetRenderer scene bbox empty", sbbox);
+            console.error("GlViewerRenderCore.resetRenderer scene bbox empty", this.scene_bbox);
             const ext = 100;
-            sbbox.expandByPoint(new RC.Vector3(-ext,-ext,-ext));
-            sbbox.expandByPoint(new RC.Vector3( ext, ext, ext));
+            this.scene_bbox.expandByPoint(new RC.Vector3(-ext,-ext,-ext));
+            this.scene_bbox.expandByPoint(new RC.Vector3( ext, ext, ext));
          }
+      }
+
+      resetRenderer()
+      {
+         this.recalcSceneBBox();
 
+         let sbbox = this.scene_bbox;
          let posV = new RC.Vector3; posV.subVectors(sbbox.max, this.rot_center);
          let negV = new RC.Vector3; negV.subVectors(sbbox.min, this.rot_center);
 
@@ -540,6 +548,19 @@ sap.ui.define([
          this.controls.update();
       }
 
+      setupCamera()
+      {
+         // To be used with JS debugger to edit the values as needed.
+
+         let pos = new RC.Vector3;
+         let lookat = new RC.Vector3;
+         let fov = 30; // in degrees
+
+         console.log("A good place to set the breakpoint and edit the values");
+
+         // Call the controller stuff, hope it's not all local, otherwise we need to edit it there.
+         // Sigh, should really have it (and RedeQuTor) in ROOT.
+      }
 
       updateViewerAttributes()
       {
@@ -641,20 +662,28 @@ sap.ui.define([
 
       //==============================================================================
 
-      request_render()
+      request_render(recalc_sbbox=false)
       {
          // console.log("REQUEST RENDER");
 
+         this.render_requested_recalc_sbbox ||= recalc_sbbox;
          if (this.render_requested) return;
          setTimeout(this.render.bind(this), 0);
          this.render_requested = true;
       }
 
       render()
       {
-         // console.log("RENDER", this.scene, this.camera, this.canvas, this.renderer);
+         console.log("RENDER", this.scene, this.camera, this.canvas, this.renderer);
 
          this.render_requested = false;
+         if (this.render_requested_recalc_sbbox) {
+            this.recalcSceneBBox();
+            this.render_requested_recalc_sbbox = false;
+         }
+         if (this.camera.isPerspectiveCamera) {
+            this.camera.optimizeNearFar(this.scene_bbox);
+         }
 
          if (this.canvas.width <= 0 || this.canvas.height <= 0) return;
 
@@ -1220,6 +1249,25 @@ sap.ui.define([
          </html>`);
          win.document.close();
       }
+
+      //==============================================================================
+      // Temporary RCore additions (to avoid updating of RCore.tgz)
+      //==============================================================================
+
+      ApplyTemporaryRCoreExtensions() {
+         console.log("GlViewerRCore.ApplyTemporaryRCoreExtensions()");
+
+         // E.g.:
+         // if (RC.PerspectiveCamera.prototype.optimizeNearFar === undefined) {
+         //    RC.PerspectiveCamera.prototype.optimizeNearFar = function(scene_bbox) {
+         //       this.matrixWorldInverse.getInverse(this.matrixWorld);
+         //       // .....
+         //    };
+         // }
+      }
+
+      //==============================================================================
+
    } // class GlViewerRCore
 
    return GlViewerRCore;