Skip to content
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

feat: add knowledge to agent #108

Merged
merged 3 commits into from
Dec 13, 2024
Merged

feat: add knowledge to agent #108

merged 3 commits into from
Dec 13, 2024

Conversation

grabbou
Copy link
Collaborator

@grabbou grabbou commented Dec 12, 2024

We separate knowledge from workflow description to avoid tasks/descriptions interfering with agent. This could possibly be something else in the future, like a method that gets it from the database/something.

This is just demonstration, as alternative to #105. I am still not sure about this.

Copy link
Collaborator

@pkarw pkarw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For me this is cool, we can merge it in

@grabbou grabbou marked this pull request as ready for review December 12, 2024 14:52
@grabbou grabbou merged commit 3626c15 into feat/refactor Dec 13, 2024
@grabbou
Copy link
Collaborator Author

grabbou commented Dec 13, 2024

I have a feeling this would be done better with tools, but I guess we can try this for now!

grabbou added a commit that referenced this pull request Dec 13, 2024
We separate knowledge from workflow description to avoid
tasks/descriptions interfering with agent. This could possibly be
something else in the future, like a method that gets it from the
database/something.

This is just demonstration, as alternative to #105. I am still not sure
about this.
grabbou added a commit that referenced this pull request Dec 13, 2024
We separate knowledge from workflow description to avoid
tasks/descriptions interfering with agent. This could possibly be
something else in the future, like a method that gets it from the
database/something.

This is just demonstration, as alternative to #105. I am still not sure
about this.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants