Skip to content

fix: correct COMPAS dataset size on index.html (60k+, not 70k+) - #528

Merged
yakew7 merged 1 commit into
yakew7:mainfrom
propcgamer20-png:fix/index-compas-record-count
Sep 9, 2026
Merged

fix: correct COMPAS dataset size on index.html (60k+, not 70k+)#528
yakew7 merged 1 commit into
yakew7:mainfrom
propcgamer20-png:fix/index-compas-record-count

Conversation

@propcgamer20-png

Copy link
Copy Markdown
Collaborator

Problem

index.html's COMPAS cards claim "70,000+ records" / "70k+ records", but COMPAS/compas-scores-raw.csv has only 60,842 data rows (wc -l = 60843 including the header). The filtered dataset the audit scripts actually use (audit.yaml row_filters: race in {African-American, Caucasian} and DisplayText == "Risk of Recidivism") is ~16.3k rows, smaller still.

Fix

Two spots updated to a figure that is accurate for the raw file shipped in the repo:

  • Project card prose: 70,000+ records -> 60,000+ records
  • Roadmap card meta: 70k+ records -> 60k+ records

60,000+ is unambiguously true for the 60,842-row file (avoids implying >=61,000).

python scripts/check_em_dash.py passes.

Closes #518

🤖 Generated with Claude Code

COMPAS/compas-scores-raw.csv has 60,842 data rows, so both the project
card ("70,000+ records") and the roadmap card ("70k+ records") overstated
the dataset size. Changed both to "60,000+" / "60k+", which is accurate
for the raw file shipped in the repo.

Closes yakew7#518

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@vercel

vercel Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

@propcgamer20-png is attempting to deploy a commit to the yashkewlani2020-gmailcom's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

@yakew7 - new PR touching a path you own, please review.

@yakew7
yakew7 merged commit 7dafe0f into yakew7:main Sep 9, 2026
17 of 18 checks passed
yakew7 added a commit that referenced this pull request Sep 9, 2026
@propcgamer20-png bumped to 33 merged PRs / 50 commits (was 19/36),
added the 14-PR batch (#528-541) to their bio, and added to 3
"Contributions by area" rows their recent work now touches (Profiler -
web, Benchmark harness, Website & explainer build).

New contributor @Aaqibhafeezkhan (2 merged PRs, first merged
2026-09-09): two one-line doc fixes (#526, #527). Added to the
Explainers and Website & explainer build area rows.

Refreshed the file's own snapshot line (was "through PR #510"; now
"through PR #541").
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.

index.html's COMPAS card overstates the dataset size ("70k+ records")

2 participants