add judgment for tos

Author: ZQlQZ

tests/test_runner.py

@@ -28,7 +28,9 @@ def _test_convert_messages(runner):
             role="user",
         )
     ]
-    actual_message = runner._convert_messages(message, session_id="test_session_id")
+    actual_message = runner._convert_messages(
+        message, session_id="test_session_id", upload_inline_data_to_tos=True
+    )
     assert actual_message == expected_message
 
     message = ["test message 1", "test message 2"]
@@ -42,7 +44,9 @@ def _test_convert_messages(runner):
             role="user",
         ),
     ]
-    actual_message = runner._convert_messages(message, session_id="test_session_id")
+    actual_message = runner._convert_messages(
+        message, session_id="test_session_id", upload_inline_data_to_tos=True
+    )
     assert actual_message == expected_message
 
 

veadk/integrations/ve_tos/ve_tos.py

@@ -15,16 +15,27 @@
 import os
 from veadk.config import getenv
 from veadk.utils.logger import get_logger
-import tos
 import asyncio
 from typing import Union
 from pydantic import BaseModel, Field
 from typing import Any
 from urllib.parse import urlparse
 from datetime import datetime
 
+# Initialize logger before using it
 logger = get_logger(__name__)
 
+# Try to import tos module, and provide helpful error message if it fails
+try:
+    import tos
+except ImportError as e:
+    logger.error(
+        "Failed to import 'tos' module. Please install it using: pip install tos\n"
+    )
+    raise ImportError(
+        "Missing 'tos' module. Please install it using: pip install tos\n"
+    ) from e
+
 
 class TOSConfig(BaseModel):
     region: str = Field(
@@ -59,10 +70,13 @@ def model_post_init(self, __context: Any) -> None:
             logger.info("Connected to TOS successfully.")
         except Exception as e:
             logger.error(f"Client initialization failed:{e}")
-            return None
+            self._client = None
 
     def create_bucket(self) -> bool:
         """If the bucket does not exist, create it"""
+        if not self._client:
+            logger.error("TOS client is not initialized")
+            return False
         try:
             self._client.head_bucket(self.config.bucket_name)
             logger.info(f"Bucket {self.config.bucket_name} already exists")
@@ -76,6 +90,9 @@ def create_bucket(self) -> bool:
                 )
                 logger.info(f"Bucket {self.config.bucket_name} created successfully")
                 return True
+            else:
+                logger.error(f"Bucket creation failed: {str(e)}")
+                return False
         except Exception as e:
             logger.error(f"Bucket creation failed: {str(e)}")
             return False
@@ -103,26 +120,24 @@ def upload(
         data: Union[str, bytes],
     ):
         if isinstance(data, str):
-            data_type = "file"
+            # data is a file path
+            return asyncio.to_thread(self._do_upload_file, object_key, data)
         elif isinstance(data, bytes):
-            data_type = "bytes"
+            # data is bytes content
+            return asyncio.to_thread(self._do_upload_bytes, object_key, data)
         else:
             error_msg = f"Upload failed: data type error. Only str (file path) and bytes are supported, got {type(data)}"
             logger.error(error_msg)
             raise ValueError(error_msg)
-        if data_type == "file":
-            return asyncio.to_thread(self._do_upload_file, object_key, data)
-        elif data_type == "bytes":
-            return asyncio.to_thread(self._do_upload_bytes, object_key, data)
 
-    def _do_upload_bytes(self, object_key: str, bytes: bytes) -> bool:
+    def _do_upload_bytes(self, object_key: str, data: bytes) -> bool:
         try:
             if not self._client:
                 return False
             if not self.create_bucket():
                 return False
             self._client.put_object(
-                bucket=self.config.bucket_name, key=object_key, content=bytes
+                bucket=self.config.bucket_name, key=object_key, content=data
             )
             logger.debug(f"Upload success, object_key: {object_key}")
             self._close()
@@ -152,6 +167,9 @@ def _do_upload_file(self, object_key: str, file_path: str) -> bool:
 
     def download(self, object_key: str, save_path: str) -> bool:
         """download image from TOS"""
+        if not self._client:
+            logger.error("TOS client is not initialized")
+            return False
         try:
             object_stream = self._client.get_object(self.config.bucket_name, object_key)
 

veadk/runner.py

@@ -87,24 +87,34 @@ def __init__(
             plugins=plugins,
         )
 
-    def _convert_messages(self, messages, session_id) -> list:
+    def _convert_messages(
+        self, messages, session_id, upload_inline_data_to_tos
+    ) -> list:
         if isinstance(messages, str):
             messages = [types.Content(role="user", parts=[types.Part(text=messages)])]
         elif isinstance(messages, MediaMessage):
             assert messages.media.endswith(".png"), (
                 "The MediaMessage only supports PNG format file for now."
             )
             data = read_png_to_bytes(messages.media)
-
-            ve_tos = VeTOS()
-            object_key, tos_url = ve_tos.build_tos_url(
-                self.user_id, self.app_name, session_id, messages.media
-            )
-            try:
-                asyncio.create_task(ve_tos.upload(object_key, data))
-            except Exception as e:
-                logger.error(f"Upload to TOS failed: {e}")
-                tos_url = None
+            tos_url = "<tos_url>"
+            if upload_inline_data_to_tos:
+                try:
+                    ve_tos = VeTOS()
+                    object_key, tos_url = ve_tos.build_tos_url(
+                        self.user_id, self.app_name, session_id, messages.media
+                    )
+                    upload_task = ve_tos.upload(object_key, data)
+                    if upload_task is not None:
+                        asyncio.create_task(upload_task)
+                except Exception as e:
+                    logger.error(f"Upload to TOS failed: {e}")
+                    tos_url = None
+
+            else:
+                logger.warning(
+                    "Loss of multimodal data may occur in the tracing process."
+                )
 
             messages = [
                 types.Content(
@@ -124,7 +134,11 @@ def _convert_messages(self, messages, session_id) -> list:
         elif isinstance(messages, list):
             converted_messages = []
             for message in messages:
-                converted_messages.extend(self._convert_messages(message, session_id))
+                converted_messages.extend(
+                    self._convert_messages(
+                        message, session_id, upload_inline_data_to_tos
+                    )
+                )
             messages = converted_messages
         else:
             raise ValueError(f"Unknown message type: {type(messages)}")
@@ -179,6 +193,7 @@ async def event_generator():
                 print()  # end with a new line
         except LlmCallsLimitExceededError as e:
             logger.warning(f"Max number of llm calls limit exceeded: {e}")
+            final_output = ""
 
         return final_output
 
@@ -189,8 +204,11 @@ async def run(
         stream: bool = False,
         run_config: RunConfig | None = None,
         save_tracing_data: bool = False,
+        upload_inline_data_to_tos: bool = False,
     ):
-        converted_messages: list = self._convert_messages(messages, session_id)
+        converted_messages: list = self._convert_messages(
+            messages, session_id, upload_inline_data_to_tos
+        )
 
         await self.short_term_memory.create_session(
             app_name=self.app_name, user_id=self.user_id, session_id=session_id
@@ -276,6 +294,7 @@ async def event_generator():
                 final_output += chunk
         except LlmCallsLimitExceededError as e:
             logger.warning(f"Max number of llm calls limit exceeded: {e}")
+            final_output = ""
 
         return final_output