Merge branch 'microsoft:main' into main

autogen/agentchat/groupchat.py

@@ -3,6 +3,9 @@
 from typing import Dict, List, Optional, Union
 from .agent import Agent
 from .conversable_agent import ConversableAgent
+import logging
+
+logger = logging.getLogger(__name__)
 
 
 @dataclass
@@ -42,6 +45,14 @@ def select_speaker_msg(self):
     def select_speaker(self, last_speaker: Agent, selector: ConversableAgent):
         """Select the next speaker."""
         selector.update_system_message(self.select_speaker_msg())
+
+        # Warn if GroupChat is underpopulated, without established changing behavior
+        n_agents = len(self.agent_names)
+        if n_agents < 3:
+            logger.warning(
+                f"GroupChat is underpopulated with {n_agents} agents. Direct communication would be more efficient."
+            )
+
         final, name = selector.generate_oai_reply(
             self.messages
             + [

autogen/code_utils.py

@@ -218,7 +218,7 @@ def execute_code(
     timeout: Optional[int] = None,
     filename: Optional[str] = None,
     work_dir: Optional[str] = None,
-    use_docker: Optional[Union[List[str], str, bool]] = docker is not None,
+    use_docker: Optional[Union[List[str], str, bool]] = True,
     lang: Optional[str] = "python",
 ) -> Tuple[int, str, str]:
     """Execute code in a docker container.
@@ -257,6 +257,15 @@ def execute_code(
         logger.error(error_msg)
         raise AssertionError(error_msg)
 
+    # Warn if docker was requested but cannot be provided. In this case
+    # the current behavior is to fall back to run natively, but this behavior
+    # is subject to change.
+    if use_docker and docker is None:
+        use_docker = False
+        logger.warning(
+            "execute_code was called with use_docker evaluating to True, but the python docker package is not available. Falling back to native code execution. Note: this fallback behavior is subject to change"
+        )
+
     timeout = timeout or DEFAULT_TIMEOUT
     original_filename = filename
     if WIN32 and lang in ["sh", "shell"] and (not use_docker):

autogen/oai/completion.py

@@ -768,6 +768,13 @@ def yes_or_no_filter(context, config, response):
         """
         if ERROR:
             raise ERROR
+
+        # Warn if a config list was provided but was empty
+        if type(config_list) is list and len(config_list) == 0:
+            logger.warning(
+                "Completion was provided with a config_list, but the list was empty. Adopting default OpenAI behavior, which reads from the 'model' parameter instead."
+            )
+
         if config_list:
             last = len(config_list) - 1
             cost = 0

autogen/oai/openai_utils.py

@@ -237,10 +237,12 @@ def config_list_from_json(
     if json_str:
         config_list = json.loads(json_str)
     else:
+        config_list_path = os.path.join(file_location, env_or_file)
         try:
-            with open(os.path.join(file_location, env_or_file)) as json_file:
+            with open(config_list_path) as json_file:
                 config_list = json.load(json_file)
         except FileNotFoundError:
+            logging.warning(f"The specified config_list file '{config_list_path}' does not exist.")
             return []
     return filter_config(config_list, filter_dict)
 

autogen/version.py

@@ -1 +1 @@
-__version__ = "0.1.8"
+__version__ = "0.1.10"