Skip to content

chore: update token badges#723

Open
johnnygreco wants to merge 4 commits into
mainfrom
johnny/chore/update-token-badages
Open

chore: update token badges#723
johnnygreco wants to merge 4 commits into
mainfrom
johnny/chore/update-token-badages

Conversation

@johnnygreco
Copy link
Copy Markdown
Contributor

@johnnygreco johnnygreco commented Jun 1, 2026

📋 Summary

Updates the README badges to reflect the latest usage and scale totals. This refreshes the generated token count, adds a purple context tokens consumed badge, adds an SDG Jobs Completed badge, and keeps the + qualifier attached to each count.

🔗 Related Issue

N/A

🔄 Changes

  • Update the README token badge from 400+ billion to 550+ billion tokens generated.
  • Add a new 2+ trillion context tokens consumed badge in purple.
  • Add a new 1+ million SDG Jobs Completed badge.

🧪 Testing

  • git diff --check passes
  • Unit tests added/updated (N/A - README badge only)
  • E2E tests added/updated (N/A - README badge only)

✅ Checklist

  • Follows commit message conventions
  • Commits are signed off (DCO)
  • Architecture docs updated (N/A - README badge only)

Signed-off-by: Johnny Greco <jogreco@nvidia.com>
@johnnygreco johnnygreco requested a review from a team as a code owner June 1, 2026 20:55
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Jun 1, 2026

Greptile Summary

This PR updates the README badges to reflect updated usage statistics for the DataDesigner project.

  • Updates the "Tokens Generated" badge from 400+ billion to 550+ billion, adds a new purple "2+ Trillion Context Tokens Consumed" badge, and adds a new "1+ Million SDG Jobs Completed" badge — all using the shields.io static badge format consistent with existing badges.

Confidence Score: 5/5

README-only change updating three static shields.io badges with no impact on code, tests, or CI.

The change touches only badge URLs in the README — no logic, configuration, or code is modified. The new badges follow the same shields.io static badge pattern already used in the file, with consistent color codes and formatting.

No files require special attention.

Important Files Changed

Filename Overview
README.md Badge-only update: token count bumped to 550B, two new static shields.io badges added for context tokens and SDG jobs; URLs and color codes are consistent with existing badge style.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[README.md Badge Row] --> B[CI Badge]
    A --> C[License Badge]
    A --> D[Python Badge]
    A --> E[NeMo Microservices Badge]
    A --> F[Code Docs Badge]
    A --> G["Tokens Generated\n400B → 550B"]
    A --> H["Context Tokens Consumed\n2T+ NEW"]
    A --> I["SDG Jobs Completed\n1M+ NEW"]
Loading

Reviews (4): Last reviewed commit: "chore: align badge plus signs" | Re-trigger Greptile

Signed-off-by: Johnny Greco <jogreco@nvidia.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 1, 2026

Code Review: PR #723 - chore: update token badges

Summary

This PR updates the README.md badges to reflect refreshed usage/scale metrics:

  • Bumps the Tokens Generated badge from 400+ Billion to 550 Billion.
  • Adds a new Context Tokens Consumed badge (2 Trillion+, purple).
  • Adds a new SDG Jobs badge (1 Million+, NVIDIA green).

Net change: +1/-1 line in README.md. Pure documentation/marketing update — no code, tests, or docs site content affected.

Findings

Minor — Inconsistent "+" suffix on the Tokens Generated badge

The original badge read 400+ Billion. The new badges added in this PR use the same convention (2 Trillion+, 1 Million+), but the updated tokens badge dropped the qualifier and now reads 550 Billion exactly:

![Tokens](.../550_Billion-Tokens_Generated-...)
![Context Tokens](.../2_Trillion+-Context_Tokens_Consumed-...)
![SDG Jobs](.../1_Million+-SDG_Jobs-...)

The PR body itself describes the change as "550 billion tokens generated" without a +, so this looks intentional, but it is visually inconsistent with the two sibling badges and with the prior 400+ framing. Consider 550_Billion+ for consistency unless 550B is meant to be an exact figure.

Style nit — Long single line

All badges are concatenated onto one Markdown line, which is already true on main; this PR extends that line further. Not a blocker (GitHub renders fine and Markdown allows it), but splitting badges one-per-line would make future diffs cleaner. Out of scope for this PR.

Code quality / correctness

  • Markdown syntax is valid; badge URLs are well-formed shields.io links.
  • Color codes match the existing palette (76b900 NVIDIA green, 8A2BE2 purple).
  • No broken links introduced.

Tests / CI

  • N/A — README-only change. Test plan in PR body correctly marks unit/E2E as not applicable.

Security

  • No secrets, no executable changes, no third-party content beyond the existing shields.io domain already in use throughout the README. No concerns.

Project conventions

  • Matches the existing badge layout pattern in README.md.
  • Commit type chore: is appropriate for a non-functional doc tweak.

Verdict

Approve with optional nit. The change is trivial, low-risk, and behaves exactly as described. Only suggestion is to consider adding + to the 550 Billion badge for visual consistency with the two new sibling badges — non-blocking.

Signed-off-by: Johnny Greco <jogreco@nvidia.com>
Signed-off-by: Johnny Greco <jogreco@nvidia.com>
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