Skip to content
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

docs(weave): document PII redaction from traces feature in Python SDK #3618

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

J2-D2-3PO
Copy link
Contributor

@J2-D2-3PO J2-D2-3PO commented Feb 6, 2025

Description

Documents PII redaction feature v1 as described in #3406.
The new page will live at https://weave-docs.wandb.ai/guides/tracking/redact-pii

Depends on

#3406

Testing

  • yarn start on local
  • SME review

@J2-D2-3PO J2-D2-3PO requested review from a team as code owners February 6, 2025 21:09
@J2-D2-3PO J2-D2-3PO self-assigned this Feb 6, 2025
@circle-job-mirror
Copy link

circle-job-mirror bot commented Feb 6, 2025

@J2-D2-3PO J2-D2-3PO marked this pull request as draft February 6, 2025 21:11
@J2-D2-3PO J2-D2-3PO changed the title docs(weave): document new PII redaction feature in Python SDK docs(weave): document PII redaction from traces feature in Python SDK Feb 6, 2025
@J2-D2-3PO
Copy link
Contributor Author

J2-D2-3PO commented Feb 6, 2025

@anandwandb @niall-wb you are both tagged for review as SMEs, please review the docs PR for tech accuracy when the feature PR is ready to ship. A few notes:

  • Comment on the PR with any requested changes / questions and I'll address.
  • Feel free to add other SME reviewers if needed.
  • Important: I used the PR description for feat(weave): PII Redaction for weave using Microsoft Presidio #3406 and internal feature doc to draft this, please help me to confirm that I'm not documenting anything not yet released and am fully documenting all user-facing functionality for v1 of this feature.

@scottire tagged you for review here too in case you want to see how this looks from the customer perspective, etc.

J2-D2-3PO

This comment was marked as outdated.

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.

1 participant