Skip to content

docs(ls): set sampling rate per client#560

Merged
Victor Moreira (victorm-lc) merged 5 commits into
mainfrom
vic/ls-per-client-sampling-rate
Sep 17, 2025
Merged

docs(ls): set sampling rate per client#560
Victor Moreira (victorm-lc) merged 5 commits into
mainfrom
vic/ls-per-client-sampling-rate

Conversation

@victorm-lc

@victorm-lc Victor Moreira (victorm-lc) commented Sep 17, 2025

Copy link
Copy Markdown
Contributor

Overview

Added example of how to set different sampling rates for tracing per client.

Type of change

Type: Update existing documentation

Checklist

Copilot AI review requested due to automatic review settings September 17, 2025 15:04
@github-actions github-actions Bot added the langsmith For docs changes to LangSmith label Sep 17, 2025

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds documentation on how to set different sampling rates for tracing per client in LangSmith, expanding beyond the existing global sampling rate configuration. The update provides practical examples for controlling trace sampling at a more granular level.

Key changes:

  • Added introductory context explaining when and why to use sampling rates
  • Enhanced the existing global sampling rate section with clearer organization
  • Added new section demonstrating per-client sampling rate configuration using Client instances and tracing_context

Comment thread src/langsmith/sample-traces.mdx
Comment thread src/langsmith/sample-traces.mdx
Comment thread src/langsmith/sample-traces.mdx
@github-actions

Copy link
Copy Markdown
Contributor

Preview ID generated: preview-viclsp-1758121517-5f7a1aa

@katmayb Kathryn May (katmayb) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just a couple small things, but lgtm!

Comment thread src/langsmith/sample-traces.mdx Outdated
Comment thread src/langsmith/sample-traces.mdx Outdated
Comment thread src/langsmith/sample-traces.mdx Outdated
Co-authored-by: Kathryn May <44557882+katmayb@users.noreply.github.com>
Copilot AI review requested due to automatic review settings September 17, 2025 15:25
Co-authored-by: Kathryn May <44557882+katmayb@users.noreply.github.com>
Co-authored-by: Kathryn May <44557882+katmayb@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@github-actions

Copy link
Copy Markdown
Contributor

Preview ID generated: preview-viclsp-1758122798-8f0a79d

@victorm-lc
Victor Moreira (victorm-lc) merged commit 14085ed into main Sep 17, 2025
12 checks passed
@victorm-lc
Victor Moreira (victorm-lc) deleted the vic/ls-per-client-sampling-rate branch September 17, 2025 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

langsmith For docs changes to LangSmith

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants