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.

Sign up for GitHub

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

Jump to bottom

Multi agent support #102

Merged
merged 8 commits into from
Mar 11, 2024
Merged

Multi agent support #102

merged 8 commits into from
Mar 11, 2024

Conversation

bboynton97
Copy link
Contributor

📥 Pull Request

  • Creates Agent decorator
    • Can be used to assign classes or functions as agents
  • Using the decorator creates an agent in the db
  • Singleton decorator
  • Makes Client adhere to a singleton pattern
  • Jupyter Notebook example

Small changes:

  • Removes frequency recording in env as it does not work on M1 chip macs
  • warn -> warning
  • print -> logging

siyangqiu
siyangqiu approved these changes Mar 8, 2024
View reviewed changes
Copy link
Contributor

@siyangqiu siyangqiu left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Some nits, but 🥳🥳🥳🥳

agentops/llm_tracker.py Outdated Show resolved Hide resolved
agentops/llm_tracker.py Outdated Show resolved Hide resolved
agentops/llm_tracker.py Outdated Show resolved Hide resolved
agentops/llm_tracker.py Outdated Show resolved Hide resolved
agentops/llm_tracker.py Show resolved Hide resolved
examples/multi_agent_example.ipynb Outdated Show resolved Hide resolved
pyproject.toml Show resolved Hide resolved
agentops/llm_tracker.py Outdated Show resolved Hide resolved
@bboynton97
Copy link
Contributor Author

  • Tracks events to agent_id now
  • Changes _ao_... to _agent_ops_...
  • Uses agent_id instead of is_agent

@bboynton97 bboynton97 merged commit 37cf12a into main Mar 11, 2024
1 check passed
@siyangqiu siyangqiu deleted the multi-agent-support branch March 21, 2024 22:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@siyangqiu siyangqiu siyangqiu approved these changes

@areibman areibman Awaiting requested review from areibman

@HowieG HowieG Awaiting requested review from HowieG

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bboynton97 @siyangqiu