Skip to content

🚀 Feature: allow disabling prompt sending as an argument to Traceloop.init() #137

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

Open
1 task done
nirga opened this issue Oct 31, 2023 · 14 comments
Open
1 task done
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@nirga
Copy link
Member

nirga commented Oct 31, 2023

Which component is this feature for?

Traceloop SDK

🔖 Feature description

Add a setting called trace_content (which will default to true) and will control whether all the instrumentations are sending sensitive content (like prompts and completions). This should override the env var behavior (similar to other flags we have).

🎤 Why is this feature needed ?

More control on this sensitive feature

✌️ How do you aim to achieve this?

🔄️ Additional Information

No response

👀 Have you spent some time to check if this feature request has been raised before?

  • I checked and didn't find similar issue

Are you willing to submit PR?

None

@nirga nirga added good first issue Good for newcomers help wanted Extra attention is needed enhancement New feature or request and removed help wanted Extra attention is needed labels Dec 15, 2023
@nirga nirga closed this as completed Jan 26, 2024
@nirga nirga reopened this Jan 26, 2024
@randolfuy01
Copy link

is this still available to work on?

@nirga
Copy link
Member Author

nirga commented Apr 10, 2024

Yes @randolfuy01 :)

@randolfuy01
Copy link

perfect I'll look into it!

@sanspareilsmyn
Copy link

@nirga @randolfuy01 Hi there! Is it still WIP? If not, I would love to start looking around this issue:)

@nirga
Copy link
Member Author

nirga commented May 21, 2024

Not sure what's @randolfuy01's status, but there are other issues that you can also look at that are unassigned @sanspareilsmyn!

@liamjdavis
Copy link

@nirga Hi! Is this issue still open? If so, I would be happy to look at it :)

@nirga
Copy link
Member Author

nirga commented Jul 17, 2024

Sounds good @liamjdavis!

@liamjdavis
Copy link

liamjdavis commented Jul 17, 2024

@nirga To clarify in greater detail,

  1. The setting is the flag in the sdk's config init.py 'is_content_tracing_enabled

def is_content_tracing_enabled() -> bool:
return (os.getenv("TRACELOOP_TRACE_CONTENT") or "true").lower() == "true"

  1. Then the setting is called in the global sdk init.py

enable_content_tracing = is_content_tracing_enabled()

  1. And I need to write the functionality that for each instrument,

instruments: Optional[Set[Instruments]] = None,

the sensitive content being shared for each individual instrument is controlled.

@nirga
Copy link
Member Author

nirga commented Jul 19, 2024

Yes!

@liamjdavis
Copy link

@nirga I've opened #1620 for this issue.

@thapasya-m
Copy link

@nirga is the issue still open to work on?

@nirga
Copy link
Member Author

nirga commented Nov 26, 2024

Yes @thapasya-m!

@shivarag200701
Copy link

@nirga is the issue still open?

@nirga
Copy link
Member Author

nirga commented May 17, 2025

Yes @shivarag200701!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

6 participants