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 back AgentOps as Optional Dependency #543

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

Conversation

bboynton97
Copy link
Contributor

@bboynton97 bboynton97 commented Apr 30, 2024

This puts AgentOps back as an optional dependency only

I also ran the black formatter to pass the github action and it made some formatting changes to files I did not touch.

# Conflicts:
#	pyproject.toml
#	src/crewai/agent.py
#	src/crewai/crew.py
#	src/crewai/tools/tool_usage.py
# Conflicts:
#	pyproject.toml
#	src/crewai/agent.py
#	src/crewai/crew.py
#	src/crewai/tools/tool_usage.py
@bboynton97 bboynton97 changed the title Make AgentOps Optional Dependency Add back AgentOps as Optional Dependency May 2, 2024
bboynton97 and others added 19 commits May 3, 2024 15:07
Changed default model value from gpt-4 to gpt-4o.
Reasoning.
gpt-4 costs 30$ per million tokens while gpt-4o costs 5$.
This is more cost friendly for default option.
Rework example so that if you use a custom LLM it doesn't throw code errors by uncommenting.
* Added timestamp to logger

Updated the logger.py file to include timestamps when logging output. For example:

 [2024-05-20 15:32:48][DEBUG]: == Working Agent: Researcher
 [2024-05-20 15:32:48][INFO]: == Starting Task: Research the topic
 [2024-05-20 15:33:22][DEBUG]: == [Researcher] Task output:

* Update tool_usage.py

* Revert "Update tool_usage.py"

This reverts commit 95d18d5.

incorrect bramch for this commit
# Conflicts:
#	src/crewai/task.py
Copy link
Collaborator

@gvieira gvieira left a comment

Choose a reason for hiding this comment

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

Quick suggestion: what if we had a generic class like AgentMonitoring that we could call across the codebase, then this class would have all the logic to optionally call agentops if it's included? Does that make sense?

@bboynton97
Copy link
Contributor Author

Quick suggestion: what if we had a generic class like AgentMonitoring that we could call across the codebase, then this class would have all the logic to optionally call agentops if it's included? Does that make sense?

We've done implementations like that for a couple other frameworks, but nothing like this exists in crew atm

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.

None yet