Filter Root (not children) (#961)

When sampling, don't randomly sample within a trace; sample whole traces
only.

js/package.json

@@ -1,6 +1,6 @@
 {
   "name": "langsmith",
-  "version": "0.1.48",
+  "version": "0.1.49",
   "description": "Client library to connect to the LangSmith LLM Tracing and Evaluation Platform.",
   "packageManager": "yarn@1.22.19",
   "files": [

js/src/client.ts

@@ -657,6 +657,11 @@ export class Client {
     } else {
       const sampled = [];
       for (const run of runs) {
+        if (run.id !== run.trace_id) {
+          sampled.push(run);
+          this.sampledPostUuids.add(run.id);
+          continue;
+        }
         if (Math.random() < this.tracingSampleRate) {
           sampled.push(run);
           this.sampledPostUuids.add(run.id);

js/src/index.ts

@@ -12,4 +12,4 @@ export type {
 export { RunTree, type RunTreeConfig } from "./run_trees.js";
 
 // Update using yarn bump-version
-export const __version__ = "0.1.48";
+export const __version__ = "0.1.49";

python/langsmith/client.py

@@ -1182,7 +1182,10 @@ def _filter_for_sampling(
         else:
             sampled = []
             for run in runs:
-                if random.random() < self.tracing_sample_rate:
+                if (
+                    run["id"] != run.get("trace_id")
+                    or random.random() < self.tracing_sample_rate
+                ):
                     sampled.append(run)
                     self._sampled_post_uuids.add(_as_uuid(run["id"]))
             return sampled

python/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "langsmith"
-version = "0.1.107"
+version = "0.1.108"
 description = "Client library to connect to the LangSmith LLM Tracing and Evaluation Platform."
 authors = ["LangChain <support@langchain.dev>"]
 license = "MIT"