-
Notifications
You must be signed in to change notification settings - Fork 18
feat: attach custom tracer w/ replicas as callback #86
Conversation
|
||
if (run.langsmith_tracer) { | ||
config.configurable ??= {}; | ||
Object.assign(config.configurable, { |
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.
Why not do this in one line with a spread operator?
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.
its a fair question. tbh just copied from above lol
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.
nitpick` this is imo noisy, which is why I opted to do that instead
config.configurable = {
...config.configurable,
langsmith_project: run.langsmith_tracer.project_name,
langsmith_example_id: run.langsmith_tracer.example_id,
}
yarn.lock
Outdated
languageName: node | ||
linkType: hard | ||
|
||
"zod@npm:^3.25.32": |
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.
Can we dedupe?
}, | ||
"devDependencies": { | ||
"@langchain/core": "^0.3.42", | ||
"@langchain/core": "^0.3.59", |
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.
Does this warrant a bump on core
?
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.
the replicas stuff won't work without the bump to core
This PR does two things:
replicas
by defining alangchain_tracer
when creating a run.