-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
refactor: reduce telemetry events count #4501
Conversation
@ZanSara Just saw this PR and remembered that we might still be sending events for each PromptTemplate loaded. We should prevent components from sending events automatically on load - if so desired. |
@vblagoje that part should be gone in telemetry v2. We're now sending events only on very limited situations. This telemetry should not be to assess absolute usage numbers, just relative ones: which docstore is more popular, which nodes are never used, etc. It doesn't try to understand how many people use the various components, just whether they're used at all. |
Proposed Changes:
Haystack imported
telemetry eventrun
call after the pipeline config has changed meaningfullyHow did you test it?
Notes for the reviewer
n/a
Checklist
fix:
,feat:
,build:
,chore:
,ci:
,docs:
,style:
,refactor:
,perf:
,test:
.