Fix an out-of-date comment for accuracy. In the OpenAI LLM service, we *don't* replace any context system messages with system instructions from the constructor.
This commit is contained in:
@@ -327,7 +327,7 @@ class BaseOpenAILLMService(LLMService):
|
|||||||
|
|
||||||
params.update(self._settings.extra)
|
params.update(self._settings.extra)
|
||||||
|
|
||||||
# Prepend system instruction from constructor, replacing any context system message
|
# Prepend system instruction from constructor
|
||||||
if self._settings.system_instruction:
|
if self._settings.system_instruction:
|
||||||
messages = params.get("messages", [])
|
messages = params.get("messages", [])
|
||||||
if messages and messages[0].get("role") == "system":
|
if messages and messages[0].get("role") == "system":
|
||||||
|
|||||||
Reference in New Issue
Block a user