Lee-W commented on code in PR #74447:
URL: https://github.com/apache/airflow/pull/74447#discussion_r4241108902
##########
providers/common/ai/src/airflow/providers/common/ai/tools/__init__.py:
##########
@@ -165,8 +165,7 @@ def collect_tools(sources: Iterable[ToolProvider |
AirflowTool]) -> list[Airflow
names = [tool.name for tool in tools]
if duplicates := sorted({name for name in names if names.count(name) > 1}):
raise ValueError(
- f"More than one tool is named {', '.join(duplicates)}. Give the
toolsets different tool "
- "name prefixes where they take one (HookToolset's
tool_name_prefix, the tool_prefix of "
- "SandboxToolset), so the model can call each of them."
+ f"More than one tool is named {', '.join(duplicates)}. "
+ "Give the toolsets different tool_prefix values so the model can
call each of them."
Review Comment:
It's now "Give the toolsets different tool_prefix values, where they take
one, so the model can call each of them."
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
