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

Add AgentOps monitoring functionality and example #1

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

bboynton97
Copy link
Collaborator

@bboynton97 bboynton97 commented Dec 11, 2023

Why are these changes needed?

This PR adds agent monitoring via (AgentOps)[https://agentops.ai].

The AssistantAgent and ConversableAgent classes are given optional constructor parameters for an AgentOps Client instance. If the instance exists, the agent records an action when sending or receiving messages between agents.

Related issue number

Checks

from autogen import AssistantAgent, UserProxyAgent, config_list_from_json
from agentops import Client

ao_client = Client(api_key=os.environ.get('AGENTOPS_API_KEY'),

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I added a placeholder to the first file, test_utils.py.

OAI_CONFIG_LIST only has variables corresponding to OpenAI. All other environment variables seem to be expected to be exported previous to running scripts.

@bboynton97 bboynton97 requested a review from areibman December 12, 2023 18:22
areibman pushed a commit that referenced this pull request Jun 7, 2024
* Switched to AzureOpenAI for api_type=="azure"

* Setting AzureOpenAI to empty object if no `openai`

* extra_ and openai_ kwargs

* test_client, support for Azure and "gpt-35-turbo-instruct"

* instruct/azure model in test_client_stream

* generalize aoai support (#1)

* generalize aoai support

* Null check, fixing tests

* cleanup test

---------

Co-authored-by: Maxim Saplin <[email protected]>

* Returning back model names for instruct

* process model in create

* None check

---------

Co-authored-by: Chi Wang <[email protected]>
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