search

i-am-bee / bee-agent-framework

The framework for building scalable agentic applications.
https://i-am-bee.github.io/bee-agent-framework/
Apache License 2.0
91 stars 22 forks source link

Wrong warning in example for code interpreter #9

Closed nheidloff closed 1 month ago

nheidloff commented 1 month ago

Just a detail. This error comes up in bee.ts even if the env variable is defined.

System Please ensure that the code interpreter is running.

The check needs to be inverse: https://github.com/i-am-bee/bee-agent-framework/blob/7ebad1fbcd5514fed3610da3025bfb2e5ea7d9e3/examples/agents/bee.ts#L51

Tomas2D commented 1 month ago

This is intended.

You can have ENV set (so PythonTool becomes activated), but the container may not be running. If the ENV is not set, then the code interpreter is not used, and thus, you can't get any error from the PythonTool (because it is not used).

nheidloff commented 1 month ago

Oh, that's a little confusing. How about changing the message to something more descriptive?

"The code interpreter is enabled. Please ensure that it is running on http://localhost:50051."

Tomas2D commented 1 month ago

Agree, updated.

Ref: https://github.com/i-am-bee/bee-agent-framework/commit/ea28b43b8a1aad3f51d191b4f7179a3672888c4f