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

Vf/operators #1793

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

venti-frappuccino
Copy link

Relates to:

#391

Risks

Low. This change only impacts bootstrap plugin and is disabled by default.

Background

Shaw had a plan in #391 to enable operator instructions, such that agents can receive goals and knowledges automatically from conversations. Further, such capability is ideally extendable by templating.

What does this PR do?

This PR basically makes the original #391 work end to end.

What kind of change is this?

Improvements (misc. changes to existing features)

  1. Add more meaningful logs to json parser.
  2. Fix the evaluator flow in Init commit of operator instructions #391 , making sure goals and knowledge can be generated and memorized.

Why are we doing this? Any context or related work?

This change enables template-based instructions for agents. FXN will iterate over it, and continue building agent-agent coordination on Eliza.

Documentation changes needed?

My changes do not require a change to the project documentation.

Testing

Where should a reviewer start?

Only plugin-bootstrap is touched thus testable. Follow the steps below if needed.

Detailed testing steps

  1. Update operators in packages/core/src/defaultCharacter.ts by pasting your user id.
  2. pnpm build && pnpm start
  3. pnpm start:client
  4. Chat with Eliza agent and look for logs in cmd, as exampled below.
Screenshot 2025-01-03 at 6 12 53 PM

Discord username

ventifrappuccino0524

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hi @venti-frappuccino! Welcome to the ai16z community. Thanks for submitting your first pull request; your efforts are helping us accelerate towards AGI. We'll review it shortly. You are now a ai16z contributor!

@venti-frappuccino venti-frappuccino changed the base branch from main to develop January 4, 2025 02:35
packages/core/src/types.ts Show resolved Hide resolved
packages/core/src/types.ts Show resolved Hide resolved
@lalalune
Copy link
Member

lalalune commented Jan 4, 2025

@venti-frappuccino this is awesome, who are you on Discord BTW haha

@venti-frappuccino
Copy link
Author

venti-frappuccino commented Jan 5, 2025

@venti-frappuccino this is awesome, who are you on Discord BTW haha

@lalalune Thank you Shaw! Probably you remember Anthony Isaacson who runs FXN. I m Anthony's friend and helping on the agent coordination. Now I become a fan of Eliza. 😄 Thanks for bringing up such a great open sourced community.

BTW ventifrappuccino0524 is my DC name.

@venti-frappuccino
Copy link
Author

venti-frappuccino commented Jan 5, 2025

Spot checked a few other PRs - seems all of them are having the integration test (installation) issue.

Raised in DC https://discord.com/channels/1253563208833433701/1300025221834739744/1325321469827350578

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.

3 participants