Skip to content

Add Nested Traces PromptFlow How-to #827

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

PeriniM
Copy link
Contributor

@PeriniM PeriniM commented Jun 2, 2025

Propagating LangSmith Traces in PromptFlow

This tutorial explains how to correctly propagate LangSmith traces in a PromptFlow application. We'll cover how to create nested traces, pass run tree information between nodes, and integrate LangGraph within a PromptFlow tool.

Copy link

vercel bot commented Jun 2, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
langsmith-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 2, 2025 2:02pm


## Introduction

This tutorial explains how to correctly propagate LangSmith traces in a [PromptFlow](https://github.com/microsoft/promptflow) application. We'll cover how to create nested traces, pass run tree information between nodes, and integrate LangGraph within a PromptFlow tool.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is too much up front. This expects the user to know about "run tree" information. I think probably better to just say like "This tutorials shows how to correctly trace a PromptFlow application in LangSmith. It covers initial setup, nested tracing, and calling the promptflow tools from a langgraph application" or something like that

return processed
```

## Creating Nested Traces
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we prefer regular sentence casing "## Creating nested traces"

return {
"processed": processed,
"langsmith_extra": {
"parent": headers
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is the extra required here??

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants