[PR #543] [CLOSED] feat: pii quickstarts no longer use recusion, added info banners #591

Closed
opened 2026-02-21 17:18:26 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langsmith-docs/pull/543
Author: @davidx33
Created: 11/21/2024
Status: Closed

Base: mainHead: pii-scrub-no-recursion


📝 Commits (7)

  • 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

📊 Changes

2 files changed (+64 additions, -123 deletions)

View changed files

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

📄 Description

changed the PII quick starts to not use recursion and have info banners stating the implementation is not exhaustive, just a general example. removing create_anonymizer code snippet temporarily while a fix is made


🔄 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/543 **Author:** [@davidx33](https://github.com/davidx33) **Created:** 11/21/2024 **Status:** ❌ Closed **Base:** `main` ← **Head:** `pii-scrub-no-recursion` --- ### 📝 Commits (7) - [`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 ### 📊 Changes **2 files changed** (+64 additions, -123 deletions) <details> <summary>View changed files</summary> 📝 `docs/observability/how_to_guides/tracing/mask_inputs_outputs.mdx` (+64 -123) 📝 `docs/observability/how_to_guides/tracing/static/presidio-anonymized.png` (+0 -0) </details> ### 📄 Description changed the PII quick starts to not use recursion and have info banners stating the implementation is not exhaustive, just a general example. removing create_anonymizer code snippet temporarily while a fix is made --- <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:26 -05:00
yindo closed this issue 2026-02-21 17:18:26 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/langsmith-docs#591