Added python code execution tool support #1470
Conversation
Signed-off-by: Ye, Xinyu <[email protected]>
XinyuYe-Intel
requested review from
lkk12014402 and
minmin-intel
as code owners
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
please add a UT for python code execution tool.
eero-t
left a comment
eero-t
left a comment
There was a problem hiding this comment.
Executor description: https://github.com/QwenLM/Qwen-Agent/blob/main/qwen_agent/tools/python_executor.py#L84
States following:
For executing python code. Not sandboxed. Do not use it for production purposes.
=> Is this going to be sandboxed?
|
We should plan and review the python code execution tool more carefully. Is this needed for v1.3 release? Shall we do a more careful design and consider other kinds of implementation not just qwen-agent? @lkk12014402 @XinyuYe-Intel |
|
RFC for this is not accepted yet (is missing necessary details): opea-project/docs#331 |
yes, we don't need depend on qwen-agent because it is not a mature code repository |
|
This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days. |
|
This PR was closed because it has been stalled for 7 days with no activity. |
Description
Added python code execution tool support.
Issues
n/a.Type of change
Dependencies
qwen-agent.
Tests
Describe the tests that you ran to verify your changes.