-
Notifications
You must be signed in to change notification settings - Fork 780
Add OpenAI integration docs #4969
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
base: v1.16
Are you sure you want to change the base?
Conversation
Signed-off-by: yaron2 <schneider.yaron@live.com>
Signed-off-by: yaron2 <schneider.yaron@live.com>
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://kind-hill-063ba6c1e-4969.westus2.5.azurestaticapps.net |
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://green-bush-04c343a1e-4969.westus2.4.azurestaticapps.net |
msfussell
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some comments after trying this out.
daprdocs/content/en/developing-applications/openai-agents/_index.md
Outdated
Show resolved
Hide resolved
| dapr run --app-id openaisessions --dapr-grpc-port 50001 --resources-path ./components -- python3 ./openai_agent.py | ||
| ``` | ||
|
|
||
| Open `http://localhost:9411` to view your the traces and dependency graph! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See [the session data stored in Redis]({{% ref "getting-started/get-started-api" %}}#step-4-see-how-the-state-is-stored-in-redis) with
hgetall "123:messages" There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You did not add this, which I really suggest
See [the session data stored in Redis]({{% ref "getting-started/get-started-api" %}}#step-4-see-how-the-state-is-stored-in-redis) with
hgetall "123:messages"
daprdocs/content/en/developing-applications/openai-agents/openai-agents-sessions.md
Outdated
Show resolved
Hide resolved
daprdocs/content/en/developing-applications/openai-agents/openai-agents-sessions.md
Outdated
Show resolved
Hide resolved
daprdocs/content/en/developing-applications/openai-agents/openai-agents-sessions.md
Outdated
Show resolved
Hide resolved
daprdocs/content/en/developing-applications/openai-agents/openai-agents-sessions.md
Outdated
Show resolved
Hide resolved
…ex.md Co-authored-by: Mark Fussell <markfussell@gmail.com> Signed-off-by: Yaron Schneider <schneider.yaron@live.com>
…ai-agents-sessions.md Co-authored-by: Mark Fussell <markfussell@gmail.com> Signed-off-by: Yaron Schneider <schneider.yaron@live.com>
…ai-agents-sessions.md Co-authored-by: Mark Fussell <markfussell@gmail.com> Signed-off-by: Yaron Schneider <schneider.yaron@live.com>
…ai-agents-sessions.md Co-authored-by: Mark Fussell <markfussell@gmail.com> Signed-off-by: Yaron Schneider <schneider.yaron@live.com>
…ai-agents-sessions.md Co-authored-by: Mark Fussell <markfussell@gmail.com> Signed-off-by: Yaron Schneider <schneider.yaron@live.com>
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://green-bush-04c343a1e-4969.westus2.4.azurestaticapps.net |
2 similar comments
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://green-bush-04c343a1e-4969.westus2.4.azurestaticapps.net |
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://green-bush-04c343a1e-4969.westus2.4.azurestaticapps.net |
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://kind-hill-063ba6c1e-4969.westus2.5.azurestaticapps.net |
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://green-bush-04c343a1e-4969.westus2.4.azurestaticapps.net |
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://kind-hill-063ba6c1e-4969.westus2.5.azurestaticapps.net |
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://green-bush-04c343a1e-4969.westus2.4.azurestaticapps.net |
msfussell
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should add the redis state store session state view command IMO
| ```bash | ||
| dapr run --app-id openaisessions --dapr-grpc-port 50001 --resources-path ./components -- python3 ./openai_agent.py | ||
| ``` | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| You can see [the session data stored in Redis]({{% ref "getting-started/get-started-api" %}}#step-4-see-how-the-state-is-stored-in-redis) with the following command | |
| ```bash | |
| hgetall "123:messages" |
Adds the documentation for how to let Dapr manage the session data for OpenAI agents with a first class integration.