From cc523e43fc9261ce2769c00aa658a05b1f121f7a Mon Sep 17 00:00:00 2001 From: garylin2099 Date: Fri, 26 Jul 2024 18:44:28 +0800 Subject: [PATCH] remove unnecessary DA tools & use clearer response at mgx_env --- metagpt/environment/mgx/mgx_env.py | 5 +++-- tests/metagpt/environment/mgx_env/run_mgx_env.py | 2 +- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/metagpt/environment/mgx/mgx_env.py b/metagpt/environment/mgx/mgx_env.py index 3a492b3ac..873358252 100644 --- a/metagpt/environment/mgx/mgx_env.py +++ b/metagpt/environment/mgx/mgx_env.py @@ -87,11 +87,12 @@ class MGXEnv(Environment): async def ask_human(self, question: str, sent_from: Role = None) -> str: # NOTE: Can be overwritten in remote setting - return await get_human_input(question) + rsp = await get_human_input(question) + return "Human response: " + rsp async def reply_to_human(self, content: str, sent_from: Role = None) -> str: # NOTE: Can be overwritten in remote setting - return "The monitor has verified the message, confirmation acknowledged. Refrain from resending duplicate messages." + return "SUCCESS, human has received your reply. Refrain from resending duplicate messages." def message_within_software_sop(self, message: Message) -> bool: # Engineer, QaEngineer can be end of the SOP. Their msg requires routing outside. diff --git a/tests/metagpt/environment/mgx_env/run_mgx_env.py b/tests/metagpt/environment/mgx_env/run_mgx_env.py index ecc2facca..b6d5341de 100644 --- a/tests/metagpt/environment/mgx_env/run_mgx_env.py +++ b/tests/metagpt/environment/mgx_env/run_mgx_env.py @@ -28,7 +28,7 @@ async def main(requirement="", enable_human_input=False, use_fixed_sop=False, al ProjectManager(use_fixed_sop=use_fixed_sop), engineer, # QaEngineer(), - DataAnalyst(tools=[""]), + DataAnalyst(), SWEAgent(), ] )