-
Notifications
You must be signed in to change notification settings - Fork 147
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: add role and tools to getting started cli experience #211
Conversation
FYI @mmurad2 - if this wasn't what you were envisioning let me know, happy to iterate on this |
this is a new one 😄 (source) |
Signed-off-by: Paul S. Schweigert <[email protected]> Fixes i-am-bee#148 We have found that bees perform best when: * they use fewer number of tools, and * they are provided explicit instructions/role Here we updated the getting started section of the README to incorporate those recommendations into the simple example. We update the system prompt to provide explicit instructions, and we remove the search tool as it isn't needed for weather-related queries. It also adds some tips related to these two points to the appropriate section. Additionally, it adds a yarn helper method to run the simple example: `yarn run quickstart` Signed-off-by: Paul S. Schweigert <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the issue was about adding role
and tools
to the CLI.
So, when someone starts the script, it first asks for tools and a role, and then the chat begins.
|
||
➡️ you can run this example after local installation, using the command `yarn start examples/agents/simple.ts` | ||
➡️ you can run this example after local installation, using the command `yarn run quickstart` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
run
is not needed
@@ -86,6 +101,8 @@ console.log(`Agent 🤖 : `, response.result.text); | |||
> | |||
> Documentation is available at https://i-am-bee.github.io/bee-agent-framework/ | |||
|
|||
➡️ See a more [advanced example](/examples/agents/bee.ts) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing .
at the end.
tools: [new OpenMeteoTool()], // for more explore 'bee-agent-framework/tools' | ||
templates: { | ||
system: BeeSystemPrompt.fork((old) => ({ | ||
...old, | ||
defaults: { | ||
instructions: | ||
"You are a helpful assistant that uses tools to answer weather-related questions.", | ||
}, | ||
})), | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This definitely should not be part of README. Keep the example as is.
Ok, I think I misunderstood the ask here. Do we currently have a way to create an agent from the command line? I believe all the examples I saw here are running an existing source code file (e.g. Would there be a way to use the API here? That seems like the ideal way to add this kind of functionality, but I'm assuming that would need the whole stack and not just the framework, so might not be an option here... |
Why would you use an API in the framework? API is built on top of the framework. Maybe consult details in the related issue. |
Signed-off-by: Paul S. Schweigert [email protected]
Which issue(s) does this pull-request address?
Closes: #148
Description
We have found that bees perform best when:
Here we updated the getting started section of the README to incorporate those recommendations into the simple example. We update the system prompt to provide explicit instructions, and we remove the search tool as it isn't needed for weather-related queries. It also adds some tips related to these two points to the appropriate section.
Additionally, it adds a yarn helper method to run the simple example:
yarn run quickstart
Checklist
yarn lint
oryarn lint:fix
yarn format
oryarn format:fix
yarn test:unit
yarn test:e2e
no tests included as this is a documentation change