-
Notifications
You must be signed in to change notification settings - Fork 779
Docs/python sdk ext langgraph #4954
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: Casper Nielsen <casper@diagrid.io>
Signed-off-by: Casper Nielsen <casper@diagrid.io>
Signed-off-by: Casper Nielsen <casper@diagrid.io>
Signed-off-by: Casper Nielsen <casper@diagrid.io>
bibryam
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.
Docs look good, but I'm not able to run the linked example
I get
dapr run --app-id langgraph-checkpointer --app-port 5001 --resources-path ./components -- python3 agent.py
ℹ️ Starting Dapr with id langgraph-checkpointer. HTTP Port: 53043. gRPC Port: 53044
== APP == Traceback (most recent call last):
== APP == File "/Users/bibryam/Projects/dapr/python-sdk/examples/langgraph-checkpointer/agent.py", line 1, in <module>
== APP == from dapr.ext.langgraph import DaprCheckpointer
== APP == ModuleNotFoundError: No module named 'dapr'
You need to install the python sdk + the ext package. The examples are part of the tests in CI where the mechanical markdown runs it and installs the dependencies. |
marcduiker
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 text suggestoins
sdkdocs/python/content/en/python-sdk-docs/python-sdk-extensions/python-langgraph.md
Outdated
Show resolved
Hide resolved
sdkdocs/python/content/en/python-sdk-docs/python-sdk-extensions/python-langgraph.md
Outdated
Show resolved
Hide resolved
sdkdocs/python/content/en/python-sdk-docs/python-sdk-extensions/python-langgraph.md
Outdated
Show resolved
Hide resolved
sdkdocs/python/content/en/python-sdk-docs/python-sdk-extensions/python-langgraph.md
Outdated
Show resolved
Hide resolved
…s/python-langgraph.md Co-authored-by: Marc Duiker <marcduiker@users.noreply.github.com> Signed-off-by: Casper Nielsen <whopsec@protonmail.com>
…s/python-langgraph.md Co-authored-by: Marc Duiker <marcduiker@users.noreply.github.com> Signed-off-by: Casper Nielsen <whopsec@protonmail.com>
…s/python-langgraph.md Co-authored-by: Marc Duiker <marcduiker@users.noreply.github.com> Signed-off-by: Casper Nielsen <whopsec@protonmail.com>
…s/python-langgraph.md Co-authored-by: Marc Duiker <marcduiker@users.noreply.github.com> Signed-off-by: Casper Nielsen <whopsec@protonmail.com>
|
Thank you @marcduiker 🚀 |
|
@CasperGN - Given the other Dapr integrations it may be best not to move this python under the new integrations section. See OpenAI docs PR |
I concur. Let's wait with this one and when Yaron's setup has been merged I'll fit this to the new structure. |
|
@CasperGN - This can now be updated |
Signed-off-by: Casper Nielsen <casper@diagrid.io>
Signed-off-by: Casper Nielsen <casper@diagrid.io>
Signed-off-by: Casper Nielsen <casper@diagrid.io>
|
@msfussell the documentation has now been moved into the new structure. The page has also been updated to reflect the same setup as the others. I took the liberty to remove "agents" from title for OpenAI as it wasn't present for crewAI and the document is anyway nested under the Agent session rendering it redundant in the header. |
Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:
In addition, please fill out the following to help reviewers understand this pull request:
Description
Adding documentation to accompany dapr/python-sdk#868
Issue reference
N/A