[Rendering] Fix bounds calculation for renderers;

Author: LittleCodingFox

Engine/Core/Rendering/Animation/SkinnedMeshRenderSystem.cs

@@ -66,6 +66,45 @@ public void Prepare()
 
     public void Preprocess((Entity, Transform, IComponent)[] entities, Camera activeCamera, Transform activeCameraTransform)
     {
+        foreach (var (_, transform, relatedComponent) in entities)
+        {
+            var renderer = relatedComponent as SkinnedMeshRenderer;
+
+            if (renderer.mesh == null ||
+                renderer.mesh.meshAsset == null ||
+                renderer.mesh.meshAssetIndex < 0 ||
+                renderer.mesh.meshAssetIndex >= renderer.mesh.meshAsset.meshes.Count ||
+                renderer.materials == null ||
+                renderer.materials.Count != renderer.mesh.submeshes.Count)
+            {
+                continue;
+            }
+
+            var skip = false;
+
+            for (var i = 0; i < renderer.materials.Count; i++)
+            {
+                if ((renderer.materials[i]?.IsValid ?? false) == false)
+                {
+                    skip = true;
+
+                    break;
+                }
+            }
+
+            if (skip)
+            {
+                continue;
+            }
+
+            if (renderer.mesh.submeshes.Count > 0 && renderer.materials.Count != renderer.mesh.submeshes.Count)
+            {
+                continue;
+            }
+
+            renderer.localBounds = renderer.mesh.bounds;
+            renderer.bounds = new AABB(transform.Position + renderer.mesh.bounds.center, renderer.mesh.bounds.extents * 2 * transform.Scale);
+        }
     }
 
     public void Process((Entity, Transform, IComponent)[] entities, Camera activeCamera, Transform activeCameraTransform, ushort viewId)

Engine/Core/Rendering/Mesh/MeshRenderSystem.cs

@@ -122,22 +122,30 @@ public void Preprocess((Entity, Transform, IComponent)[] entities, Camera active
         {
             var r = relatedComponent as MeshRenderer;
 
-            if (r.isVisible == false ||
-                r.mesh == null ||
+            if (r.mesh == null ||
                 r.materials == null ||
                 r.materials.Count == 0)
             {
                 continue;
             }
 
+            var skip = false;
+
             for (var i = 0; i < r.materials.Count; i++)
             {
-                if (r.materials[i]?.IsValid == false)
+                if ((r.materials[i]?.IsValid ?? false) == false)
                 {
-                    continue;
+                    skip = true;
+
+                    break;
                 }
             }
 
+            if(skip)
+            {
+                continue;
+            }
+
             if (r.mesh.submeshes.Count > 0 && r.materials.Count != r.mesh.submeshes.Count)
             {
                 continue;

Engine/Core/Rendering/Sprites/SpriteRenderSystem.cs

@@ -99,11 +99,6 @@ public void Preprocess((Entity, Transform, IComponent)[] entities,
         {
             var r = relatedComponent as SpriteRenderer;
 
-            if(r.isVisible == false)
-            {
-                continue;
-            }
-
             var hasValidAnimation = r.animation != null &&
                 r.animation.texture != null &&
                 r.animation.texture.Disposed == false &&