Add sorted function in required fields to avoid randomness

modelscope · Feb 26, 2024 · 8e8ea2c · 8e8ea2c
1 parent 4d4899b
commit 8e8ea2c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/agentscope/service/service_factory.py b/src/agentscope/service/service_factory.py
@@ -121,7 +121,9 @@ def get(
             ),
         )
 
-        args_required = list(set(args_agent) - set(args_defaults.keys()))
+        args_required = sorted(
+            list(set(args_agent) - set(args_defaults.keys())),
+        )
 
         # Prepare types of the arguments, remove the return type
         args_types = {

diff --git a/tests/service_factory_test.py b/tests/service_factory_test.py
@@ -126,7 +126,7 @@ def func(  # type: ignore
                             "b": {},
                             "a": {"type": "string"},
                         },
-                        "required": ["b", "a"],
+                        "required": ["a", "b"],
                     },
                 },
             },