From 71571f3dfae4c13b750a936fc4c84c724cee0833 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E0=AE=AE=E0=AE=A9=E0=AF=8B=E0=AE=9C=E0=AF=8D=E0=AE=95?= =?UTF-8?q?=E0=AF=81=E0=AE=AE=E0=AE=BE=E0=AE=B0=E0=AF=8D=20=E0=AE=AA?= =?UTF-8?q?=E0=AE=B4=E0=AE=A9=E0=AE=BF=E0=AE=9A=E0=AF=8D=E0=AE=9A=E0=AE=BE?= =?UTF-8?q?=E0=AE=AE=E0=AE=BF?= Date: Mon, 16 Sep 2024 18:08:23 +0530 Subject: [PATCH] swap system role for o1 models fixes https://github.com/All-Hands-AI/OpenHands/issues/3883 --- agenthub/codeact_agent/codeact_agent.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/agenthub/codeact_agent/codeact_agent.py b/agenthub/codeact_agent/codeact_agent.py index cd4615993b25..71d10b76653a 100644 --- a/agenthub/codeact_agent/codeact_agent.py +++ b/agenthub/codeact_agent/codeact_agent.py @@ -5,6 +5,7 @@ from agenthub.codeact_agent.action_parser import CodeActResponseParser from openhands.controller.agent import Agent from openhands.controller.state.state import State +from openhands.core import config2 from openhands.core.config import AgentConfig, load_app_config from openhands.core.logger import openhands_logger as logger from openhands.core.message import ImageContent, Message, TextContent @@ -250,9 +251,10 @@ def step(self, state: State) -> Action: return self.action_parser.parse(response) def _get_messages(self, state: State) -> list[Message]: + system_role = 'user' if 'o1-' in config2.model else 'system' messages: list[Message] = [ Message( - role='system', + role=system_role, content=[ TextContent( text=self.prompt_manager.system_message,