langchain-ai / langgraphjs

⚡ Build language agents as graphs ⚡
https://langchain-ai.github.io/langgraphjs/
MIT License
353 stars 54 forks source link

Langsmith Traces not nesting as expected #176

Open sanjiv-upgiant opened 2 months ago

sanjiv-upgiant commented 2 months ago

Issue #1. How my langsmith tracing looks with new changes in @langchain/core library. Isn't the RunnableSequence part of langgraph node itself? Why is that separate row in langsmith?

image

Issue #2. You guys updated the packages, but there are no working way of actually updating graph State properly in any of the example. The documentation is old , entry point has changed and reducer with default values is acting weird. Not sure what reducer is even doing.

hinthornw commented 1 month ago

Thanks for reporting, @sanjiv-upgiant ! And thanks for your patience.

I updates a lot of the docs since you've created this so not sure if (2) is still applicable - there's a couple more docs left for me to recompile, but all the how-tos worked in deno and typescript when i ran them last

For (1), do you have a minimal code example to reproduce this? Hard to help without.

Also next time if you have 2 issues to report, could you please create two separate issues/tickets? Otherwise we are prone to close one prematurely or leave one open that has already been addressed. Thanks!

I'll leave this open for now until you can confirm either way. Will close in due time if I don't here other issues/responses. Thanks again!

chentschel commented 3 weeks ago

@hinthornw Im experiencing the same issue with langsmith only in with after a build (i.e vercel preview.). All seems ok in local dev.

Screenshot 2024-07-03 at 5 43 13 PM

In the local env it shows nested in a correct way.

Screenshot 2024-07-03 at 5 45 04 PM