added lod support for .asset type meshes

Author: dooly123

Basis/Packages/com.basis.sdk/Scripts/Editor/BasisBuildBlendshapeStripper.cs

@@ -58,10 +58,7 @@ public MeshRemapInfo(SkinnedMeshRenderer renderer, Mesh oldMesh, Mesh newMesh, I
     /// - Then we remap BasisAvatar indices to the new meshes.
     /// - Finally, callbacks fire so other systems can remap too.
     /// </summary>
-    public static void StripForBuild(
-        BasisAssetBundleObject settings,
-        GameObject buildRoot,
-        Basis.Scripts.BasisSdk.BasisAvatar avatarOnClone)
+    public static void StripForBuild(BasisAssetBundleObject settings, GameObject buildRoot,Basis.Scripts.BasisSdk.BasisAvatar avatarOnClone)
     {
         if (buildRoot == null) throw new ArgumentNullException(nameof(buildRoot));
         if (avatarOnClone == null) throw new ArgumentNullException(nameof(avatarOnClone));
@@ -133,12 +130,12 @@ public static void StripForBuild(
             }
             catch (Exception ex)
             {
-                Debug.LogException(ex);
+                BasisDebug.LogError($"{ex.Message} {ex.StackTrace}");
             }
         }
 
         AssetDatabase.SaveAssets();
-        AssetDatabase.Refresh();
+        AssetDatabase.Refresh( ImportAssetOptions.ForceSynchronousImport);
 
         // Remap default avatar indices (by name)
         UpdateAvatarBlendshapeIndicesAfterStrip(
@@ -156,7 +153,7 @@ public static void StripForBuild(
         }
         catch (Exception ex)
         {
-            Debug.LogException(ex);
+            BasisDebug.LogError($"{ex.Message} {ex.StackTrace}");
         }
     }
 

Basis/Packages/com.basis.sdk/Scripts/Editor/SDKInspector/BasisAvatarSDKInspector.cs

@@ -262,7 +262,7 @@ public void SetupItems()
         faceVisemeMeshField.allowSceneObjects = true;
         AvatarIconField.allowSceneObjects = true;
 
-      //  AvatarIconField.value = null;
+        //  AvatarIconField.value = null;
         animatorField.value = Avatar.Animator;
         faceBlinkMeshField.value = Avatar.FaceBlinkMesh;
         faceVisemeMeshField.value = Avatar.FaceVisemeMesh;
@@ -382,6 +382,8 @@ private async void EventCallbackAvatarBundle(List<BuildTarget> targets, Texture2
             // Add the result label to the UI
             uiElementsRoot.Add(resultLabel);
             //  BuildReportViewerWindow.ShowWindow();
+
+            GameObject.Destroy(buildRoot);
         }
         else
         {
@@ -395,109 +397,104 @@ private async void EventCallbackAvatarBundle(List<BuildTarget> targets, Texture2
             }
         }
     }
-#if UNITY_6000_2_OR_NEWER
-    /// <summary>
-    /// Generate Mesh LODs via ModelImporter for all SkinnedMeshRenderers under the given root.
-    /// Requires Unity 6000.2+ where ModelImporter.generateMeshLods exists.
-    /// </summary>
-    /// <param name="root">Root GameObject (e.g., your avatar/prefab in the scene or a prefab asset loaded in memory).</param>
-    /// <param name="lodLimit">
-    /// Maximum mesh LOD to generate. Use -1 to leave the current importer value unchanged.
-    /// </param>
+#if UNITY_6000_2_OR_NEWER && UNITY_EDITOR
     public void GenerateMeshLODs(int lodLimit = -1)
     {
         var smrs = Avatar.GetComponentsInChildren<SkinnedMeshRenderer>(true);
         if (smrs == null || smrs.Length == 0)
         {
-            Debug.LogWarning($"GenerateMeshLODs: No SkinnedMeshRenderer found under.");
+            Debug.LogWarning("GenerateMeshLODs: No SkinnedMeshRenderer found under root.");
+            return;
         }
 
-        // Collect unique importer asset paths (FBX/OBJ). Multiple meshes often come from the same file.
-        var pathsNeedingReimport = new HashSet<string>();
+        var modelPathsNeedingReimport = new HashSet<string>();
+        var meshAssetsNeedingSave = new HashSet<Mesh>(); // for .asset meshes (and others without ModelImporter)
 
         foreach (var smr in smrs)
         {
-            if (smr == null || smr.sharedMesh == null)
-                continue;
+            if (smr == null || smr.sharedMesh == null) continue;
 
-            // Get the asset path for the mesh; for model sub-assets this is the FBX/OBJ path.
-            string meshPath = AssetDatabase.GetAssetPath(smr.sharedMesh);
-            if (string.IsNullOrEmpty(meshPath))
-                continue;
+            var mesh = smr.sharedMesh;
+            string meshPath = AssetDatabase.GetAssetPath(mesh);
+            if (string.IsNullOrEmpty(meshPath)) continue;
 
             var importer = AssetImporter.GetAtPath(meshPath) as ModelImporter;
-            if (importer == null)
-                continue; // Not a model-imported mesh (e.g., .asset mesh), skip.
-
-            // Set importer flags
-            bool changed = false;
 
-            if (!importer.generateMeshLods)
+            if (importer != null)
             {
-                importer.generateMeshLods = true;
-                changed = true;
-            }
+                bool changed = false;
+
+                if (!importer.generateMeshLods)
+                {
+                    importer.generateMeshLods = true;
+                    changed = true;
+                }
 
-            if (lodLimit >= 0 && importer.maximumMeshLod != lodLimit)
+                if (lodLimit >= 0 && importer.maximumMeshLod != lodLimit)
+                {
+                    importer.maximumMeshLod = lodLimit;
+                    changed = true;
+                }
+
+                if (changed)
+                {
+                    modelPathsNeedingReimport.Add(meshPath);
+                }
+            }
+            else
             {
-                importer.maximumMeshLod = lodLimit;
-                changed = true;
+                // .asset Mesh (or otherwise not model-imported):
+                // Generate Mesh LODs directly on the mesh asset.
+                // meshLodLimit here = "max number of LOD levels to generate" (see docs).
+                // 0 => generate none beyond original; negative => auto-stop at ~64 indices.  :contentReference[oaicite:3]{index=3}
+                int meshLodLimit = lodLimit; // reuse your parameter; adjust semantics if you want
+                MeshLodUtility.GenerateMeshLods(mesh, meshLodLimit);
+
+                EditorUtility.SetDirty(mesh);
+                meshAssetsNeedingSave.Add(mesh);
             }
 
-            if (changed)
-                pathsNeedingReimport.Add(meshPath);
-
-            // Component-level preferences (do not require reimport)
+            // Renderer-level knobs (work once mesh actually has Mesh LODs) :contentReference[oaicite:4]{index=4}
             smr.meshLodSelectionBias = 0f;
             smr.forceMeshLod = -1;
         }
 
-        if (pathsNeedingReimport.Count == 0)
+        // Reimport model files (FBX/OBJ)
+        if (modelPathsNeedingReimport.Count > 0)
         {
-            Debug.Log("GenerateMeshLODs: No importer changes detected.");
-            return;
-        }
-
-        try
-        {
-            AssetDatabase.StartAssetEditing();
-
-            int i = 0;
-            int total = pathsNeedingReimport.Count;
-            foreach (var path in pathsNeedingReimport)
+            try
             {
-                if (EditorUtility.DisplayCancelableProgressBar(
-                        "Reimporting Models (LODs)",
-                        $"{i + 1}/{total}: {path}",
-                        (float)i / total))
-                {
-                    Debug.LogWarning("GenerateMeshLODs: Canceled by user.");
-                    break;
-                }
+                AssetDatabase.StartAssetEditing();
+                int i = 0, total = modelPathsNeedingReimport.Count;
 
-                var importer = AssetImporter.GetAtPath(path) as ModelImporter;
-                if (importer != null)
+                foreach (var path in modelPathsNeedingReimport)
                 {
-                    // Write settings and reimport this asset immediately.
-                    // Either approach works; SaveAndReimport is the simplest.
-                    importer.SaveAndReimport();
-                    // Alternatively:
-                    // AssetDatabase.WriteImportSettingsIfDirty(path);
-                    // AssetDatabase.ImportAsset(path, ImportAssetOptions.ForceUpdate);
+                    if (EditorUtility.DisplayCancelableProgressBar(
+                            "Reimporting Models (Mesh LODs)",
+                            $"{i + 1}/{total}: {path}",
+                            (float)i / total))
+                    {
+                        Debug.LogWarning("GenerateMeshLODs: Canceled by user.");
+                        break;
+                    }
+
+                    var mi = AssetImporter.GetAtPath(path) as ModelImporter;
+                    if (mi != null) mi.SaveAndReimport();
+                    i++;
                 }
-
-                i++;
+            }
+            finally
+            {
+                EditorUtility.ClearProgressBar();
+                AssetDatabase.StopAssetEditing();
             }
         }
-        finally
-        {
-            EditorUtility.ClearProgressBar();
-            AssetDatabase.StopAssetEditing();
+
+        // Save modified .asset meshes
+        if (meshAssetsNeedingSave.Count > 0)
             AssetDatabase.SaveAssets();
-            // No need for AssetDatabase.Refresh(); reimports were explicit.
-        }
 
-        Debug.Log($"GenerateMeshLODs: Reimported {pathsNeedingReimport.Count} model asset(s).");
+        Debug.Log($"GenerateMeshLODs: Reimported {modelPathsNeedingReimport.Count} model asset(s), updated {meshAssetsNeedingSave.Count} mesh asset(s).");
     }
 #endif
     public void AvatarTestInEditorClickFunction()