langchain-ai/langsmith-docs
1
0
Fork 0
mirror of https://github.com/langchain-ai/langsmith-docs.git synced 2026-08-27 01:41:19 -04:00
Code Issues 16 Packages Projects Releases Wiki Activity

[PR #549] [MERGED] Pii scrub no recursion #595

New Issue
Closed
opened 2026-02-21 17:18:27 -05:00 by yindo · 0 comments
yindo commented 2026-02-21 17:18:27 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langsmith-docs/pull/549
Author: @davidx33
Created: 11/25/2024
Status: ✅ Merged
Merged: 11/25/2024
Merged by: @davidx33

Base: main ← Head: pii-scrub-no-recursion

📝 Commits (8)

  • ede4808 new presidio anonymized screenshot
  • 7f83ab3 new code that does not use recursion, has an info pop up
  • e8255bc correct initialization
  • 9467167 standardize comments
  • 36917b0 delete unneeded commments
  • c9f3ea3 delete unneeded comment from regex function
  • 31c7293 standardize use of content
  • 3ad2e2c cleaner implementation for checking whether message exists. fixing create anonymizer

📊 Changes

2 files changed (+70 additions, -65 deletions)

View changed files

📝 docs/observability/how_to_guides/tracing/mask_inputs_outputs.mdx (+70 -65)
📝 docs/observability/how_to_guides/tracing/static/presidio-anonymized.png (+0 -0)

📄 Description

changed pii quickstarts to no longer recursively call presidio/comprehend. added info banners. corrected create_anonymizer

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/langsmith-docs/pull/549 **Author:** [@davidx33](https://github.com/davidx33) **Created:** 11/25/2024 **Status:** ✅ Merged **Merged:** 11/25/2024 **Merged by:** [@davidx33](https://github.com/davidx33) **Base:** `main` ← **Head:** `pii-scrub-no-recursion` --- ### 📝 Commits (8) - [`ede4808`](https://github.com/langchain-ai/langsmith-docs/commit/ede48081793aa0dec904c547ab80b9f883f7a2f7) new presidio anonymized screenshot - [`7f83ab3`](https://github.com/langchain-ai/langsmith-docs/commit/7f83ab336ee0dc8490412ee962d37a92a2b71c67) new code that does not use recursion, has an info pop up - [`e8255bc`](https://github.com/langchain-ai/langsmith-docs/commit/e8255bc8a95fe7c560dd92ad190ccd90596f46bc) correct initialization - [`9467167`](https://github.com/langchain-ai/langsmith-docs/commit/9467167f4373bb6a9c0611916fdc0f2e562e37ee) standardize comments - [`36917b0`](https://github.com/langchain-ai/langsmith-docs/commit/36917b0ff92a9fac433fefeb75d11e23ac306c4b) delete unneeded commments - [`c9f3ea3`](https://github.com/langchain-ai/langsmith-docs/commit/c9f3ea3db52f8039493666216bdb8ef770a14b89) delete unneeded comment from regex function - [`31c7293`](https://github.com/langchain-ai/langsmith-docs/commit/31c7293a987ddee7ad720a2ee1854d901cf1d8e1) standardize use of content - [`3ad2e2c`](https://github.com/langchain-ai/langsmith-docs/commit/3ad2e2c5937b3881d42e859e7d1c176689f80bb0) cleaner implementation for checking whether message exists. fixing create anonymizer ### 📊 Changes **2 files changed** (+70 additions, -65 deletions) <details> <summary>View changed files</summary> 📝 `docs/observability/how_to_guides/tracing/mask_inputs_outputs.mdx` (+70 -65) 📝 `docs/observability/how_to_guides/tracing/static/presidio-anonymized.png` (+0 -0) </details> ### 📄 Description changed pii quickstarts to no longer recursively call presidio/comprehend. added info banners. corrected create_anonymizer --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-21 17:18:27 -05:00
yindo closed this issue 2026-02-21 17:18:27 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
pull-request
Mirrored from GitHub Pull Request
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/langsmith-docs#595
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?