Skip to content

docs: Updated telemetry disclosure#595

Merged
kirit93 merged 2 commits into
mainfrom
kirit93/telemetry-disclosure
May 1, 2026
Merged

docs: Updated telemetry disclosure#595
kirit93 merged 2 commits into
mainfrom
kirit93/telemetry-disclosure

Conversation

@kirit93
Copy link
Copy Markdown
Contributor

@kirit93 kirit93 commented May 1, 2026

Updated telemetry details

@kirit93 kirit93 requested a review from a team as a code owner May 1, 2026 17:18
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 1, 2026

Docs preview: https://22f6e6e8.dd-docs-preview.pages.dev

Notebook tutorials are placeholder-only in previews.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 1, 2026

Greptile Summary

This PR refactors the telemetry disclosure in README.md: the brief intro section is simplified and now links to a new, more comprehensive "Telemetry & privacy" section added at the end of the document, which covers data collected, opt-out mechanics, and third-party endpoint privacy practices. The top-models chart and its date range/last-updated stamp are refreshed to reflect YTD data through May 1, 2026.

Confidence Score: 5/5

Documentation-only change with no code impact; safe to merge.

No logic, code, or security issues are present. Changes are limited to README prose restructuring and a refreshed binary image asset.

No files require special attention.

Important Files Changed

Filename Overview
README.md Telemetry section condensed with a link to a new detailed "Telemetry & privacy" section; chart date range updated to YTD 1/1/2026–5/1/2026 and last-updated stamp refreshed to May 1, 2026.
docs/images/top-models.png Binary chart image updated to reflect expanded YTD date range through May 1, 2026.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[User reads README] --> B[Telemetry section - brief summary]
    B --> C{Opt out?}
    C -- "Set NEMO_TELEMETRY_ENABLED=false" --> D[Telemetry disabled]
    C -- "Keep default" --> E[Telemetry enabled]
    E --> F[Collects model names + token counts]
    F --> G[Aggregated data shared with community]
    B --> H["More details → #telemetry-and-privacy"]
    H --> I[Telemetry & privacy section]
    I --> J[Scope: NeMo Data Designer library only]
    I --> K["Third-party endpoints (e.g. NVIDIA Build) governed by their own ToS"]
Loading

Reviews (2): Last reviewed commit: "Updated chart to YTD" | Re-trigger Greptile

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kirit93 – I'm thinking we should use a wider time window for these charts. It's too dominated by the current research project we are working on.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated to YTD.

@kirit93 kirit93 merged commit c119f8f into main May 1, 2026
50 checks passed
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.

2 participants