@babylonjs/core 5.23.0 → 5.25.0

package/scene.js

@@ -25,7 +25,7 @@ import { UniqueIdGenerator } from "./Misc/uniqueIdGenerator.js";
 import { ReadFile, RequestFile, LoadFile } from "./Misc/fileTools.js";
 import { LightConstants } from "./Lights/lightConstants.js";
 import { ComputePressureObserverWrapper } from "./Misc/computePressure.js";
-import { ArrayTools } from "./Misc/arrayTools.js";
+import { _ObserveArray } from "./Misc/arrayTools.js";
 /**
  * Define how the scene should favor performance over ease of use
  */
@@ -51,13 +51,13 @@ export class Scene extends AbstractScene {
     constructor(engine, options) {
         super();
         // Members
-        /** @hidden */
+        /** @internal */
         this._inputManager = new InputManager(this);
         /** Define this parameter if you are using multiple cameras and you want to specify which one should be used for pointer position */
         this.cameraToUseForPointers = null;
-        /** @hidden */
+        /** @internal */
         this._isScene = true;
-        /** @hidden */
+        /** @internal */
         this._blockEntityCollection = false;
         /**
          * Gets or sets a boolean that indicates if the scene must clear the render buffer before rendering a frame
@@ -328,7 +328,7 @@ export class Scene extends AbstractScene {
          */
         this.onAnimationFileImportedObservable = new Observable();
         // Animations
-        /** @hidden */
+        /** @internal */
         this._registeredForLateAnimationBindings = new SmartArrayNoDuplicate(256);
         /**
          * Gets or sets a boolean indicating if the user want to entirely skip the picking phase when a pointer move event occurs.
@@ -401,7 +401,7 @@ export class Scene extends AbstractScene {
         // Lights
         this._shadowsEnabled = true;
         this._lightsEnabled = true;
-        this._activeCameras = ArrayTools.MakeObservableArray(this.onActiveCamerasChanged, []);
+        this._unObserveActiveCameras = null;
         // Textures
         this._texturesEnabled = true;
         // Physics
@@ -473,13 +473,13 @@ export class Scene extends AbstractScene {
         this.proceduralTexturesEnabled = true;
         // Performance counters
         this._totalVertices = new PerfCounter();
-        /** @hidden */
+        /** @internal */
         this._activeIndices = new PerfCounter();
-        /** @hidden */
+        /** @internal */
         this._activeParticles = new PerfCounter();
-        /** @hidden */
+        /** @internal */
         this._activeBones = new PerfCounter();
-        /** @hidden */
+        /** @internal */
         this._animationTime = 0;
         /**
          * Gets or sets a general scale for animation speed
@@ -493,10 +493,10 @@ export class Scene extends AbstractScene {
         this._defaultFrameBufferCleared = false;
         this._viewUpdateFlag = -1;
         this._projectionUpdateFlag = -1;
-        /** @hidden */
+        /** @internal */
         this._toBeDisposed = new Array(256);
         this._activeRequests = new Array();
-        /** @hidden */
+        /** @internal */
         this._pendingData = new Array();
         this._isDisposed = false;
         /**
@@ -508,11 +508,11 @@ export class Scene extends AbstractScene {
         this._processedMaterials = new SmartArray(256);
         this._renderTargets = new SmartArrayNoDuplicate(256);
         this._materialsRenderTargets = new SmartArrayNoDuplicate(256);
-        /** @hidden */
+        /** @internal */
         this._activeParticleSystems = new SmartArray(256);
         this._activeSkeletons = new SmartArrayNoDuplicate(32);
         this._softwareSkinnedMeshes = new SmartArrayNoDuplicate(32);
-        /** @hidden */
+        /** @internal */
         this._activeAnimatables = new Array();
         this._transformMatrix = Matrix.Zero();
         /**
@@ -521,12 +521,12 @@ export class Scene extends AbstractScene {
          */
         this.requireLightSorting = false;
         /**
-         * @hidden
+         * @internal
          * Backing store of defined scene components.
          */
         this._components = [];
         /**
-         * @hidden
+         * @internal
          * Backing store of defined scene components.
          */
         this._serializableComponents = [];
@@ -535,112 +535,112 @@ export class Scene extends AbstractScene {
          */
         this._transientComponents = [];
         /**
-         * @hidden
+         * @internal
          * Defines the actions happening before camera updates.
          */
         this._beforeCameraUpdateStage = Stage.Create();
         /**
-         * @hidden
+         * @internal
          * Defines the actions happening before clear the canvas.
          */
         this._beforeClearStage = Stage.Create();
         /**
-         * @hidden
+         * @internal
          * Defines the actions happening before clear the canvas.
          */
         this._beforeRenderTargetClearStage = Stage.Create();
         /**
-         * @hidden
+         * @internal
          * Defines the actions when collecting render targets for the frame.
          */
         this._gatherRenderTargetsStage = Stage.Create();
         /**
-         * @hidden
+         * @internal
          * Defines the actions happening for one camera in the frame.
          */
         this._gatherActiveCameraRenderTargetsStage = Stage.Create();
         /**
-         * @hidden
+         * @internal
          * Defines the actions happening during the per mesh ready checks.
          */
         this._isReadyForMeshStage = Stage.Create();
         /**
-         * @hidden
+         * @internal
          * Defines the actions happening before evaluate active mesh checks.
          */
         this._beforeEvaluateActiveMeshStage = Stage.Create();
         /**
-         * @hidden
+         * @internal
          * Defines the actions happening during the evaluate sub mesh checks.
          */
         this._evaluateSubMeshStage = Stage.Create();
         /**
-         * @hidden
+         * @internal
          * Defines the actions happening during the active mesh stage.
          */
         this._preActiveMeshStage = Stage.Create();
         /**
-         * @hidden
+         * @internal
          * Defines the actions happening during the per camera render target step.
          */
         this._cameraDrawRenderTargetStage = Stage.Create();
         /**
-         * @hidden
+         * @internal
          * Defines the actions happening just before the active camera is drawing.
          */
         this._beforeCameraDrawStage = Stage.Create();
         /**
-         * @hidden
+         * @internal
          * Defines the actions happening just before a render target is drawing.
          */
         this._beforeRenderTargetDrawStage = Stage.Create();
         /**
-         * @hidden
+         * @internal
          * Defines the actions happening just before a rendering group is drawing.
          */
         this._beforeRenderingGroupDrawStage = Stage.Create();
         /**
-         * @hidden
+         * @internal
          * Defines the actions happening just before a mesh is drawing.
          */
         this._beforeRenderingMeshStage = Stage.Create();
         /**
-         * @hidden
+         * @internal
          * Defines the actions happening just after a mesh has been drawn.
          */
         this._afterRenderingMeshStage = Stage.Create();
         /**
-         * @hidden
+         * @internal
          * Defines the actions happening just after a rendering group has been drawn.
          */
         this._afterRenderingGroupDrawStage = Stage.Create();
         /**
-         * @hidden
+         * @internal
          * Defines the actions happening just after the active camera has been drawn.
          */
         this._afterCameraDrawStage = Stage.Create();
         /**
-         * @hidden
+         * @internal
          * Defines the actions happening just after a render target has been drawn.
          */
         this._afterRenderTargetDrawStage = Stage.Create();
         /**
-         * @hidden
+         * @internal
          * Defines the actions happening just after rendering all cameras and computing intersections.
          */
         this._afterRenderStage = Stage.Create();
         /**
-         * @hidden
+         * @internal
          * Defines the actions happening when a pointer move event happens.
          */
         this._pointerMoveStage = Stage.Create();
         /**
-         * @hidden
+         * @internal
          * Defines the actions happening when a pointer down event happens.
          */
         this._pointerDownStage = Stage.Create();
         /**
-         * @hidden
+         * @internal
          * Defines the actions happening when a pointer up event happens.
          */
         this._pointerUpStage = Stage.Create();
@@ -657,11 +657,11 @@ export class Scene extends AbstractScene {
             length: 0
         };
         this._preventFreeActiveMeshesAndRenderingGroups = false;
-        /** @hidden */
+        /** @internal */
         this._activeMeshesFrozen = false;
         this._activeMeshesFrozenButKeepClipping = false;
         this._skipEvaluateActiveMeshesCompletely = false;
-        /** @hidden */
+        /** @internal */
         this._allowPostProcessClearColor = true;
         /**
          * User updatable function that will return a deterministic frame time when engine is in deterministic lock step mode
@@ -680,6 +680,7 @@ export class Scene extends AbstractScene {
          * Note: Compute pressure is an experimental API.
          */
         this.onComputePressureChanged = new Observable();
+        this.activeCameras = new Array();
         const fullOptions = {
             useGeometryUniqueIdsMap: true,
             useMaterialMeshMap: true,
@@ -927,7 +928,7 @@ export class Scene extends AbstractScene {
      * @param effect The effect to be bound
      * @param variableName name of the shader variable that will hold the eye position
      * @param isVector3 true to indicates that variableName is a Vector3 and not a Vector4
-     * @return the computed eye position
+     * @returns the computed eye position
      */
     bindEyePosition(effect, variableName = "vEyePosition", isVector3 = false) {
         const eyePosition = this._forcedViewPosition
@@ -1067,7 +1068,16 @@ export class Scene extends AbstractScene {
         return this._activeCameras;
     }
     set activeCameras(cameras) {
-        this._activeCameras = ArrayTools.MakeObservableArray(this.onActiveCamerasChanged, cameras);
+        if (this._unObserveActiveCameras) {
+            this._unObserveActiveCameras();
+            this._unObserveActiveCameras = null;
+        }
+        if (cameras) {
+            this._unObserveActiveCameras = _ObserveArray(cameras, () => {
+                this.onActiveCamerasChanged.notifyObservers(this);
+            });
+        }
+        this._activeCameras = cameras;
     }
     /** Gets or sets the current active camera */
     get activeCamera() {
@@ -1117,7 +1127,7 @@ export class Scene extends AbstractScene {
     get skeletonsEnabled() {
         return this._skeletonsEnabled;
     }
-    /** @hidden */
+    /** @internal */
     get collisionCoordinator() {
         if (!this._collisionCoordinator) {
             this._collisionCoordinator = Scene.CollisionCoordinatorFactory();
@@ -1144,7 +1154,7 @@ export class Scene extends AbstractScene {
         }
     }
     /**
-     * @hidden
+     * @internal
      * Add a component to the scene.
      * Note that the ccomponent could be registered on th next frame if this is called after
      * the register component stage.
@@ -1159,7 +1169,7 @@ export class Scene extends AbstractScene {
         }
     }
     /**
-     * @hidden
+     * @internal
      * Gets a component from the scene.
      * @param name defines the name of the component to retrieve
      * @returns the component or null if not present
@@ -1180,7 +1190,7 @@ export class Scene extends AbstractScene {
         return "Scene";
     }
     /**
-     * @hidden
+     * @internal
      */
     _getDefaultMeshCandidates() {
         this._defaultMeshCandidates.data = this.meshes;
@@ -1188,8 +1198,7 @@ export class Scene extends AbstractScene {
         return this._defaultMeshCandidates;
     }
     /**
-     * @param mesh
-     * @hidden
+     * @internal
      */
     _getDefaultSubMeshCandidates(mesh) {
         this._defaultSubMeshCandidates.data = mesh.subMeshes;
@@ -1419,7 +1428,7 @@ export class Scene extends AbstractScene {
      * This function will check if the scene can be rendered (textures are loaded, shaders are compiled)
      * Delay loaded resources are not taking in account
      * @param checkRenderTargets true to also check that the meshes rendered as part of a render target are ready (default: true)
-     * @return true if all required resources are ready
+     * @returns true if all required resources are ready
      */
     isReady(checkRenderTargets = true) {
         if (this._isDisposed) {
@@ -1647,8 +1656,7 @@ export class Scene extends AbstractScene {
         });
     }
     /**
-     * @param checkRenderTargets
-     * @hidden
+     * @internal
      */
     _checkIsReady(checkRenderTargets = false) {
         this._registerTransientComponents();
@@ -2229,7 +2237,7 @@ export class Scene extends AbstractScene {
     /**
      * sets the active camera of the scene using its Id
      * @param id defines the camera's Id
-     * @return the new active camera or null if none found.
+     * @returns the new active camera or null if none found.
      */
     setActiveCameraById(id) {
         const camera = this.getCameraById(id);
@@ -2255,7 +2263,7 @@ export class Scene extends AbstractScene {
     /**
      * get an animation group using its name
      * @param name defines the material's name
-     * @return the animation group or null if none found.
+     * @returns the animation group or null if none found.
      */
     getAnimationGroupByName(name) {
         for (let index = 0; index < this.animationGroups.length; index++) {
@@ -2268,7 +2276,7 @@ export class Scene extends AbstractScene {
     /**
      * Get a material using its unique id
      * @param uniqueId defines the material's unique id
-     * @return the material or null if none found.
+     * @returns the material or null if none found.
      */
     getMaterialByUniqueID(uniqueId) {
         for (let index = 0; index < this.materials.length; index++) {
@@ -2281,7 +2289,7 @@ export class Scene extends AbstractScene {
     /**
      * get a material using its id
      * @param id defines the material's Id
-     * @return the material or null if none found.
+     * @returns the material or null if none found.
      */
     getMaterialById(id) {
         for (let index = 0; index < this.materials.length; index++) {
@@ -2295,7 +2303,7 @@ export class Scene extends AbstractScene {
      * Gets a the last added material using a given id
      * @param id defines the material's Id
      * @param allowMultiMaterials determines whether multimaterials should be considered
-     * @return the last material with the given id or null if none found.
+     * @returns the last material with the given id or null if none found.
      */
     getLastMaterialById(id, allowMultiMaterials = false) {
         for (let index = this.materials.length - 1; index >= 0; index--) {
@@ -2315,7 +2323,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a material using its name
      * @param name defines the material's name
-     * @return the material or null if none found.
+     * @returns the material or null if none found.
      */
     getMaterialByName(name) {
         for (let index = 0; index < this.materials.length; index++) {
@@ -2328,7 +2336,7 @@ export class Scene extends AbstractScene {
     /**
      * Get a texture using its unique id
      * @param uniqueId defines the texture's unique id
-     * @return the texture or null if none found.
+     * @returns the texture or null if none found.
      */
     getTextureByUniqueId(uniqueId) {
         for (let index = 0; index < this.textures.length; index++) {
@@ -2341,7 +2349,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a texture using its name
      * @param name defines the texture's name
-     * @return the texture or null if none found.
+     * @returns the texture or null if none found.
      */
     getTextureByName(name) {
         for (let index = 0; index < this.textures.length; index++) {
@@ -2380,7 +2388,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a camera using its name
      * @param name defines the camera's name
-     * @return the camera or null if none found.
+     * @returns the camera or null if none found.
      */
     getCameraByName(name) {
         for (let index = 0; index < this.cameras.length; index++) {
@@ -2393,7 +2401,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a bone using its Id
      * @param id defines the bone's Id
-     * @return the bone or null if not found
+     * @returns the bone or null if not found
      */
     getBoneById(id) {
         for (let skeletonIndex = 0; skeletonIndex < this.skeletons.length; skeletonIndex++) {
@@ -2409,7 +2417,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a bone using its id
      * @param name defines the bone's name
-     * @return the bone or null if not found
+     * @returns the bone or null if not found
      */
     getBoneByName(name) {
         for (let skeletonIndex = 0; skeletonIndex < this.skeletons.length; skeletonIndex++) {
@@ -2425,7 +2433,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a light node using its name
      * @param name defines the the light's name
-     * @return the light or null if none found.
+     * @returns the light or null if none found.
      */
     getLightByName(name) {
         for (let index = 0; index < this.lights.length; index++) {
@@ -2438,7 +2446,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a light node using its Id
      * @param id defines the light's Id
-     * @return the light or null if none found.
+     * @returns the light or null if none found.
      */
     getLightById(id) {
         for (let index = 0; index < this.lights.length; index++) {
@@ -2451,7 +2459,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a light node using its scene-generated unique Id
      * @param uniqueId defines the light's unique Id
-     * @return the light or null if none found.
+     * @returns the light or null if none found.
      */
     getLightByUniqueId(uniqueId) {
         for (let index = 0; index < this.lights.length; index++) {
@@ -2464,7 +2472,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a particle system by Id
      * @param id defines the particle system Id
-     * @return the corresponding system or null if none found
+     * @returns the corresponding system or null if none found
      */
     getParticleSystemById(id) {
         for (let index = 0; index < this.particleSystems.length; index++) {
@@ -2477,7 +2485,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a geometry using its Id
      * @param id defines the geometry's Id
-     * @return the geometry or null if none found.
+     * @returns the geometry or null if none found.
      */
     getGeometryById(id) {
         for (let index = 0; index < this.geometries.length; index++) {
@@ -2507,7 +2515,7 @@ export class Scene extends AbstractScene {
      * Add a new geometry to this scene
      * @param geometry defines the geometry to be added to the scene.
      * @param force defines if the geometry must be pushed even if a geometry with this id already exists
-     * @return a boolean defining if the geometry was added or not
+     * @returns a boolean defining if the geometry was added or not
      */
     pushGeometry(geometry, force) {
         if (!force && this._getGeometryByUniqueId(geometry.uniqueId)) {
@@ -2520,7 +2528,7 @@ export class Scene extends AbstractScene {
     /**
      * Removes an existing geometry
      * @param geometry defines the geometry to be removed from the scene
-     * @return a boolean defining if the geometry was removed or not
+     * @returns a boolean defining if the geometry was removed or not
      */
     removeGeometry(geometry) {
         let index;
@@ -2562,7 +2570,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets the first added mesh found of a given Id
      * @param id defines the Id to search for
-     * @return the mesh found or null if not found at all
+     * @returns the mesh found or null if not found at all
      */
     getMeshById(id) {
         for (let index = 0; index < this.meshes.length; index++) {
@@ -2585,7 +2593,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets the first added transform node found of a given Id
      * @param id defines the Id to search for
-     * @return the found transform node or null if not found at all.
+     * @returns the found transform node or null if not found at all.
      */
     getTransformNodeById(id) {
         for (let index = 0; index < this.transformNodes.length; index++) {
@@ -2598,7 +2606,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a transform node with its auto-generated unique Id
      * @param uniqueId defines the unique Id to search for
-     * @return the found transform node or null if not found at all.
+     * @returns the found transform node or null if not found at all.
      */
     getTransformNodeByUniqueId(uniqueId) {
         for (let index = 0; index < this.transformNodes.length; index++) {
@@ -2621,7 +2629,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a mesh with its auto-generated unique Id
      * @param uniqueId defines the unique Id to search for
-     * @return the found mesh or null if not found at all.
+     * @returns the found mesh or null if not found at all.
      */
     getMeshByUniqueId(uniqueId) {
         for (let index = 0; index < this.meshes.length; index++) {
@@ -2634,7 +2642,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a the last added mesh using a given Id
      * @param id defines the Id to search for
-     * @return the found mesh or null if not found at all.
+     * @returns the found mesh or null if not found at all.
      */
     getLastMeshById(id) {
         for (let index = this.meshes.length - 1; index >= 0; index--) {
@@ -2647,7 +2655,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a the last added node (Mesh, Camera, Light) using a given Id
      * @param id defines the Id to search for
-     * @return the found node or null if not found at all
+     * @returns the found node or null if not found at all
      */
     getLastEntryById(id) {
         let index;
@@ -2676,7 +2684,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a node (Mesh, Camera, Light) using a given Id
      * @param id defines the Id to search for
-     * @return the found node or null if not found at all
+     * @returns the found node or null if not found at all
      */
     getNodeById(id) {
         const mesh = this.getMeshById(id);
@@ -2704,7 +2712,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a node (Mesh, Camera, Light) using a given name
      * @param name defines the name to search for
-     * @return the found node or null if not found at all.
+     * @returns the found node or null if not found at all.
      */
     getNodeByName(name) {
         const mesh = this.getMeshByName(name);
@@ -2732,7 +2740,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a mesh using a given name
      * @param name defines the name to search for
-     * @return the found mesh or null if not found at all.
+     * @returns the found mesh or null if not found at all.
      */
     getMeshByName(name) {
         for (let index = 0; index < this.meshes.length; index++) {
@@ -2745,7 +2753,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a transform node using a given name
      * @param name defines the name to search for
-     * @return the found transform node or null if not found at all.
+     * @returns the found transform node or null if not found at all.
      */
     getTransformNodeByName(name) {
         for (let index = 0; index < this.transformNodes.length; index++) {
@@ -2758,7 +2766,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a skeleton using a given Id (if many are found, this function will pick the last one)
      * @param id defines the Id to search for
-     * @return the found skeleton or null if not found at all.
+     * @returns the found skeleton or null if not found at all.
      */
     getLastSkeletonById(id) {
         for (let index = this.skeletons.length - 1; index >= 0; index--) {
@@ -2771,7 +2779,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a skeleton using a given auto generated unique id
      * @param  uniqueId defines the unique id to search for
-     * @return the found skeleton or null if not found at all.
+     * @returns the found skeleton or null if not found at all.
      */
     getSkeletonByUniqueId(uniqueId) {
         for (let index = 0; index < this.skeletons.length; index++) {
@@ -2784,7 +2792,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a skeleton using a given id (if many are found, this function will pick the first one)
      * @param id defines the id to search for
-     * @return the found skeleton or null if not found at all.
+     * @returns the found skeleton or null if not found at all.
      */
     getSkeletonById(id) {
         for (let index = 0; index < this.skeletons.length; index++) {
@@ -2797,7 +2805,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a skeleton using a given name
      * @param name defines the name to search for
-     * @return the found skeleton or null if not found at all.
+     * @returns the found skeleton or null if not found at all.
      */
     getSkeletonByName(name) {
         for (let index = 0; index < this.skeletons.length; index++) {
@@ -2810,7 +2818,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a morph target manager  using a given id (if many are found, this function will pick the last one)
      * @param id defines the id to search for
-     * @return the found morph target manager or null if not found at all.
+     * @returns the found morph target manager or null if not found at all.
      */
     getMorphTargetManagerById(id) {
         for (let index = 0; index < this.morphTargetManagers.length; index++) {
@@ -2823,7 +2831,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a morph target using a given id (if many are found, this function will pick the first one)
      * @param id defines the id to search for
-     * @return the found morph target or null if not found at all.
+     * @returns the found morph target or null if not found at all.
      */
     getMorphTargetById(id) {
         for (let managerIndex = 0; managerIndex < this.morphTargetManagers.length; ++managerIndex) {
@@ -2840,7 +2848,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a morph target using a given name (if many are found, this function will pick the first one)
      * @param name defines the name to search for
-     * @return the found morph target or null if not found at all.
+     * @returns the found morph target or null if not found at all.
      */
     getMorphTargetByName(name) {
         for (let managerIndex = 0; managerIndex < this.morphTargetManagers.length; ++managerIndex) {
@@ -2857,7 +2865,7 @@ export class Scene extends AbstractScene {
     /**
      * Gets a post process using a given name (if many are found, this function will pick the first one)
      * @param name defines the name to search for
-     * @return the found post process or null if not found at all.
+     * @returns the found post process or null if not found at all.
      */
     getPostProcessByName(name) {
         for (let postProcessIndex = 0; postProcessIndex < this.postProcesses.length; ++postProcessIndex) {
@@ -2891,7 +2899,7 @@ export class Scene extends AbstractScene {
      * If you don't care and just want to get the data no matter what, use the more convenient getOrAddExternalDataWithFactory() method.
      * @param key the unique key that identifies the data
      * @param data the data object to associate to the key for this Engine instance
-     * @return true if no such key were already present and the data was added successfully, false otherwise
+     * @returns true if no such key were already present and the data was added successfully, false otherwise
      */
     addExternalData(key, data) {
         if (!this._externalData) {
@@ -2902,7 +2910,7 @@ export class Scene extends AbstractScene {
     /**
      * Get an externally attached data from its key
      * @param key the unique key that identifies the data
-     * @return the associated data, if present (can be null), or undefined if not present
+     * @returns the associated data, if present (can be null), or undefined if not present
      */
     getExternalData(key) {
         if (!this._externalData) {
@@ -2914,7 +2922,7 @@ export class Scene extends AbstractScene {
      * Get an externally attached data from its key, create it using a factory if it's not already present
      * @param key the unique key that identifies the data
      * @param factory the factory that will be called to create the instance if and only if it doesn't exists
-     * @return the associated data, can be null if the factory returned null.
+     * @returns the associated data, can be null if the factory returned null.
      */
     getOrAddExternalDataWithFactory(key, factory) {
         if (!this._externalData) {
@@ -2925,7 +2933,7 @@ export class Scene extends AbstractScene {
     /**
      * Remove an externally attached data from the Engine instance
      * @param key the unique key that identifies the data
-     * @return true if the data was successfully removed, false if it doesn't exist
+     * @returns true if the data was successfully removed, false if it doesn't exist
      */
     removeExternalData(key) {
         return this._externalData.remove(key);
@@ -3018,7 +3026,7 @@ export class Scene extends AbstractScene {
             }
         }
     }
-    /** @hidden */
+    /** @internal */
     _isInIntermediateRendering() {
         return this._intermediateRendering;
     }
@@ -3279,10 +3287,7 @@ export class Scene extends AbstractScene {
         }
     }
     /**
-     * @param camera
-     * @param rigParent
-     * @param bindFrameBuffer
-     * @hidden
+     * @internal
      */
     _renderForCamera(camera, rigParent, bindFrameBuffer = true) {
         if (camera && camera._skipRendering) {
@@ -3460,13 +3465,12 @@ export class Scene extends AbstractScene {
         }
     }
     /**
-     * @param step
-     * @hidden
+     * @internal
      */
     _advancePhysicsEngineStep(step) {
         // Do nothing. Code will be replaced if physics engine component is referenced
     }
-    /** @hidden */
+    /** @internal */
     _animate() {
         // Nothing to do as long as Animatable have not been imported.
     }
@@ -3752,6 +3756,7 @@ export class Scene extends AbstractScene {
             this.activeCamera._activeMeshes.dispose();
             this.activeCamera = null;
         }
+        this.activeCameras = null;
         this._activeMeshes.dispose();
         this._renderingManager.dispose();
         this._processedMaterials.dispose();
@@ -3835,7 +3840,8 @@ export class Scene extends AbstractScene {
         this._disposeList(this.meshes, (item) => item.dispose(true));
         this._disposeList(this.transformNodes, (item) => item.dispose(true));
         // Release cameras
-        this._disposeList(this.cameras);
+        const cameras = this.cameras;
+        this._disposeList(cameras);
         // Release materials
         if (this._defaultMaterial) {
             this._defaultMaterial.dispose();
@@ -4070,7 +4076,7 @@ export class Scene extends AbstractScene {
         return this._inputManager.getPointerOverMesh();
     }
     // Misc.
-    /** @hidden */
+    /** @internal */
     _rebuildGeometries() {
         for (const geometry of this.geometries) {
             geometry._rebuild();
@@ -4093,7 +4099,7 @@ export class Scene extends AbstractScene {
             }
         }
     }
-    /** @hidden */
+    /** @internal */
     _rebuildTextures() {
         for (const texture of this.textures) {
             texture._rebuild();
@@ -4229,14 +4235,7 @@ export class Scene extends AbstractScene {
         }
     }
     /**
-     * @param fileOrUrl
-     * @param onSuccess
-     * @param onProgress
-     * @param useOfflineSupport
-     * @param useArrayBuffer
-     * @param onError
-     * @param onOpened
-     * @hidden
+     * @internal
      */
     _loadFile(fileOrUrl, onSuccess, onProgress, useOfflineSupport, useArrayBuffer, onError, onOpened) {
         const request = LoadFile(fileOrUrl, onSuccess, onProgress, useOfflineSupport ? this.offlineProvider : undefined, useArrayBuffer, onError, onOpened);
@@ -4247,12 +4246,7 @@ export class Scene extends AbstractScene {
         return request;
     }
     /**
-     * @param fileOrUrl
-     * @param onProgress
-     * @param useOfflineSupport
-     * @param useArrayBuffer
-     * @param onOpened
-     * @hidden
+     * @internal
      */
     _loadFileAsync(fileOrUrl, onProgress, useOfflineSupport, useArrayBuffer, onOpened) {
         return new Promise((resolve, reject) => {
@@ -4264,14 +4258,7 @@ export class Scene extends AbstractScene {
         });
     }
     /**
-     * @param url
-     * @param onSuccess
-     * @param onProgress
-     * @param useOfflineSupport
-     * @param useArrayBuffer
-     * @param onError
-     * @param onOpened
-     * @hidden
+     * @internal
      */
     _requestFile(url, onSuccess, onProgress, useOfflineSupport, useArrayBuffer, onError, onOpened) {
         const request = RequestFile(url, onSuccess, onProgress, useOfflineSupport ? this.offlineProvider : undefined, useArrayBuffer, onError, onOpened);
@@ -4282,12 +4269,7 @@ export class Scene extends AbstractScene {
         return request;
     }
     /**
-     * @param url
-     * @param onProgress
-     * @param useOfflineSupport
-     * @param useArrayBuffer
-     * @param onOpened
-     * @hidden
+     * @internal
      */
     _requestFileAsync(url, onProgress, useOfflineSupport, useArrayBuffer, onOpened) {
         return new Promise((resolve, reject) => {
@@ -4299,12 +4281,7 @@ export class Scene extends AbstractScene {
         });
     }
     /**
-     * @param file
-     * @param onSuccess
-     * @param onProgress
-     * @param useArrayBuffer
-     * @param onError
-     * @hidden
+     * @internal
      */
     _readFile(file, onSuccess, onProgress, useArrayBuffer, onError) {
         const request = ReadFile(file, onSuccess, onProgress, useArrayBuffer, onError);
@@ -4315,10 +4292,7 @@ export class Scene extends AbstractScene {
         return request;
     }
     /**
-     * @param file
-     * @param onProgress
-     * @param useArrayBuffer
-     * @hidden
+     * @internal
      */
     _readFileAsync(file, onProgress, useArrayBuffer) {
         return new Promise((resolve, reject) => {
@@ -4356,8 +4330,7 @@ Scene.MinDeltaTime = 1.0;
  */
 Scene.MaxDeltaTime = 1000.0;
 /**
- * @param id
- * @hidden
+ * @internal
  */
 Scene.prototype.setActiveCameraByID = function (id) {
     return this.setActiveCameraById(id);