[Android] Use AlertDialog instead of Toast (mlc-ai#1039)

Author: cyx-6

android/MLCChat/app/src/main/java/ai/mlc/mlcchat/AppViewModel.kt

@@ -2,6 +2,9 @@ package ai.mlc.mlcchat
 
 import ai.mlc.mlcllm.ChatModule
 import android.app.Application
+import android.content.ClipData
+import android.content.ClipboardManager
+import android.content.Context
 import android.os.Environment
 import android.widget.Toast
 import androidx.compose.runtime.mutableStateOf
@@ -23,6 +26,8 @@ class AppViewModel(application: Application) : AndroidViewModel(application) {
     val modelList = emptyList<ModelState>().toMutableStateList()
     val chatState = ChatState()
     val modelSampleList = emptyList<ModelRecord>().toMutableStateList()
+    private var showAlert = mutableStateOf(false)
+    private var alertMessage = mutableStateOf("")
     private var appConfig = AppConfig(
         emptyList(),
         emptyList<ModelRecord>().toMutableList(),
@@ -44,25 +49,46 @@ class AppViewModel(application: Application) : AndroidViewModel(application) {
         loadAppConfig()
     }
 
+    fun supportedModelLibs(): List<String> {
+        return appConfig.modelLibs
+    }
+
+    fun isShowingAlert(): Boolean {
+        return showAlert.value
+    }
+
+    fun errorMessage(): String {
+        return alertMessage.value
+    }
+
+    fun dismissAlert() {
+        require(showAlert.value)
+        showAlert.value = false
+    }
+
+    fun copyError() {
+        require(showAlert.value)
+        val clipboard =
+            application.getSystemService(Context.CLIPBOARD_SERVICE) as ClipboardManager
+        clipboard.setPrimaryClip(ClipData.newPlainText("MLCChat", errorMessage()))
+    }
+
+    private fun issueAlert(error: String) {
+        showAlert.value = true
+        alertMessage.value = error
+    }
+
     fun requestAddModel(url: String, localId: String?) {
         if (localId != null && localIdSet.contains(localId)) {
-            Toast.makeText(
-                application,
-                "localId: $localId has been occupied",
-                Toast.LENGTH_SHORT
-            ).show()
+            issueAlert("localId: $localId has been occupied")
         } else {
             downloadModelConfig(if (url.endsWith("/")) url else "$url/", localId, false)
         }
     }
 
     fun requestDeleteModel(localId: String) {
         deleteModel(localId)
-        Toast.makeText(
-            application,
-            "Model: $localId has been deleted",
-            Toast.LENGTH_SHORT
-        ).show()
+        issueAlert("Model: $localId has been deleted")
     }
 
 
@@ -133,11 +159,7 @@ class AppViewModel(application: Application) : AndroidViewModel(application) {
     private fun isModelConfigAllowed(modelConfig: ModelConfig): Boolean {
         if (appConfig.modelLibs.contains(modelConfig.modelLib)) return true;
         viewModelScope.launch {
-            Toast.makeText(
-                application,
-                "Model lib ${modelConfig.modelLib} is not supported.",
-                Toast.LENGTH_SHORT
-            ).show()
+            issueAlert("Model lib ${modelConfig.modelLib} is not supported.")
         }
         return false
     }
@@ -169,11 +191,7 @@ class AppViewModel(application: Application) : AndroidViewModel(application) {
                         }
                         if (localIdSet.contains(modelConfig.localId)) {
                             tempFile.delete()
-                            Toast.makeText(
-                                application,
-                                "${modelConfig.localId} has been used, please consider another local ID",
-                                Toast.LENGTH_SHORT
-                            ).show()
+                            issueAlert("${modelConfig.localId} has been used, please consider another local ID")
                             return@launch
                         }
                         if (!isModelConfigAllowed(modelConfig)) {
@@ -188,21 +206,13 @@ class AppViewModel(application: Application) : AndroidViewModel(application) {
                         addModelConfig(modelConfig, modelUrl, isBuiltin)
                     } catch (e: Exception) {
                         viewModelScope.launch {
-                            Toast.makeText(
-                                application,
-                                "Add model failed: ${e.localizedMessage}",
-                                Toast.LENGTH_SHORT
-                            ).show()
+                            issueAlert("Add model failed: ${e.localizedMessage}")
                         }
                     }
                 }
             } catch (e: Exception) {
                 viewModelScope.launch {
-                    Toast.makeText(
-                        application,
-                        "Download model config failed: ${e.localizedMessage}",
-                        Toast.LENGTH_SHORT
-                    ).show()
+                    issueAlert("Download model config failed: ${e.localizedMessage}")
                 }
             }
 

android/MLCChat/app/src/main/java/ai/mlc/mlcchat/StartView.kt

@@ -20,6 +20,7 @@ import androidx.compose.material.icons.outlined.Delete
 import androidx.compose.material.icons.outlined.Download
 import androidx.compose.material.icons.outlined.Pause
 import androidx.compose.material.icons.outlined.Schedule
+import androidx.compose.material3.AlertDialog
 import androidx.compose.material3.Divider
 import androidx.compose.material3.ExperimentalMaterial3Api
 import androidx.compose.material3.Icon
@@ -94,9 +95,14 @@ fun StartView(
                 }
             }
             if (isAddingModel) {
-                Text(
-                    text = "Add Model Variant", modifier = Modifier.padding(top = 10.dp)
-                )
+                Text(text = "Supported Base Model Libs", modifier = Modifier.padding(top = 10.dp))
+                for (lib in appViewModel.supportedModelLibs()) {
+                    Text(
+                        text = lib,
+                        style = MaterialTheme.typography.bodyMedium
+                    )
+                }
+                Text(text = "Add Model Variant", modifier = Modifier.padding(top = 10.dp))
                 LazyColumn() {
                     items(
                         items = appViewModel.modelSampleList
@@ -148,10 +154,36 @@ fun StartView(
                 }
             }
         }
-
+        if (appViewModel.isShowingAlert()) {
+            AlertDialog(
+                onDismissRequest = { appViewModel.dismissAlert() },
+                onConfirmation = { appViewModel.copyError() },
+                error = appViewModel.errorMessage()
+            )
+        }
     }
 }
 
+@ExperimentalMaterial3Api
+@Composable
+fun AlertDialog(
+    onDismissRequest: () -> Unit,
+    onConfirmation: () -> Unit,
+    error: String,
+) {
+    AlertDialog(
+        title = { Text(text = "Error") },
+        text = { Text(text = error) },
+        onDismissRequest = { onDismissRequest() },
+        confirmButton = {
+            TextButton(onClick = { onConfirmation() }) { Text("Copy") }
+        },
+        dismissButton = {
+            TextButton(onClick = { onDismissRequest() }) { Text("Dismiss") }
+        }
+    )
+}
+
 @Composable
 fun ModelView(
     navController: NavController,

android/prepare_libs.sh

@@ -9,7 +9,10 @@ python prepare_model_lib.py
 
 cd build
 touch config.cmake
-echo "set(TVM_HOME ${TVM_HOME})" >> config.cmake
+if [ ${TVM_HOME-0} -ne 0 ]; then
+  echo "set(TVM_HOME ${TVM_HOME})" >> config.cmake
+fi
+
 cmake .. \
       -DCMAKE_BUILD_TYPE=Release \
       -DCMAKE_TOOLCHAIN_FILE=${ANDROID_NDK}/build/cmake/android.toolchain.cmake \