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

OpenBB agent copilot #15

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

OpenBB agent copilot #15

wants to merge 8 commits into from

Conversation

DidierRLopes
Copy link
Contributor

image

It leverages the OpenBB Agent repo: https://github.com/OpenBB-finance/openbb-agents

Copy link
Collaborator

@mnicstruwig mnicstruwig left a comment

Choose a reason for hiding this comment

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

A couple of tweaks 🙏 .

openbb-agent-copilot/README.md Outdated Show resolved Hide resolved
openbb-agent-copilot/README.md Outdated Show resolved Hide resolved
try:
result = openbb_agent(str(chat_messages), verbose=False, openbb_pat=os.getenv("OPENBB_PAT"))

return {"output": result}
Copy link
Collaborator

Choose a reason for hiding this comment

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

You'll still need to stream the result back (just as a single chunk) using the create_message_stream function. We now use named events for our SSEs for things to be parsed and rendered correctly on the front-end.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I actually tried this and had no success with it, can you share if we have more documentation on how to do this? Because in this case the output is literally just a string.

I know we have an async openbb_agent but I looked at the repo, and it's missing the PAT argument which is really important as it grants it access to all my data.

PS: I kept getting that the following screenshot. cc @mnicstruwig

Screenshot 2024-08-20 at 12 57 51 AM

Copy link
Collaborator

Choose a reason for hiding this comment

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

@DidierRLopes I think there might be something I've missed on my part. Let me investigate for you, and I'll give you an update 🙏 .

Copy link
Collaborator

Choose a reason for hiding this comment

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

@DidierRLopes Fixed the outputs using SSEs -- we just had to use the hasStreaming=true in Copilot (since Terminal Pro officially only supports streaming output now). We just stream back the entire answer in a single chunk.

openbb-agent-copilot/openbb_agent_copilot/main.py Outdated Show resolved Hide resolved
openbb-agent-copilot/openbb_agent_copilot/main.py Outdated Show resolved Hide resolved
openbb-agent-copilot/openbb_agent_copilot/models.py Outdated Show resolved Hide resolved
@@ -3,7 +3,7 @@
"name": "OpenBB Agent Copilot",
"description": "AI financial analyst using the OpenBB Platform.",
"image": "https://github.com/user-attachments/assets/010d7590-0a65-4b3f-b21a-0cbc0d95bcb9",
"hasStreaming": false,
"hasStreaming": true,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

AHHHHH - this might have been the culprit.

Thank you!

@DidierRLopes DidierRLopes changed the title OpenBB agent copilot - WIP OpenBB agent copilot Aug 28, 2024
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