-
Notifications
You must be signed in to change notification settings - Fork 137
Feat/ext langgraph checkpoint #868
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
Conversation
…or consistincy Signed-off-by: Casper Nielsen <casper@diagrid.io>
…port Signed-off-by: Casper Nielsen <casper@diagrid.io>
Signed-off-by: Casper Nielsen <casper@diagrid.io>
Signed-off-by: Casper Nielsen <casper@diagrid.io>
…ites 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>
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>
fab03ca to
7869a64
Compare
Signed-off-by: Casper Nielsen <casper@diagrid.io>
Signed-off-by: Casper Nielsen <casper@diagrid.io>
Signed-off-by: Casper Nielsen <casper@diagrid.io>
|
@CasperGN Please have a look at other examples in the
|
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>
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>
Signed-off-by: Casper Nielsen <casper@diagrid.io>
|
@acroca I've done the above and verified it runs locally with the |
Signed-off-by: Casper Nielsen <casper@diagrid.io>
|
@acroca can you try to trigger CI to see if pulling a small llama model to be used for tests is too slow? |
Signed-off-by: Casper Nielsen <casper@diagrid.io>
Signed-off-by: Casper Nielsen <casper@diagrid.io>
Signed-off-by: Casper Nielsen <casper@diagrid.io>
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #868 +/- ##
==========================================
+ Coverage 86.63% 87.04% +0.41%
==========================================
Files 84 100 +16
Lines 4473 6629 +2156
==========================================
+ Hits 3875 5770 +1895
- Misses 598 859 +261 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Signed-off-by: Casper Nielsen <casper@diagrid.io>
Signed-off-by: Casper Nielsen <casper@diagrid.io>
…Add sleep to ensure ollama us serving before pulling model Signed-off-by: Casper Nielsen <casper@diagrid.io>
Signed-off-by: Casper Nielsen <casper@diagrid.io>
|
@acroca do you want me to add the conditional run for the ext packages and ollama in this PR or should I rather add this in a follow up? I'll also need to do a follow up to implement the delete function similar to the changes in this one |
This PR adds roughly 2.5min to the CI build, right? I think it'd be good to split in a different runs, but I also want to experiment with running each validation in a different step to run. |
Yeah and considering we'd be carrying more ext- packages in we should also have a conditional check on these so we don't have to pull ollama + llama3.2 every single time but only when the ext packages that needs it requires testing |
Description
Extending
dapr-ext-langgraphto expand on the handling forput,put_writesandget_tupleto conform with the native LangGraphRedisSaver. This ensures that the demo's presented in the LangGraph Academy are functional.Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: N/A
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list: