Skip to content

Fix GAAP misspellings in earnings21 NLP reference files#54

Merged
migueljette merged 1 commit into
mainfrom
fix/e21-gaap-misspellings
May 29, 2026
Merged

Fix GAAP misspellings in earnings21 NLP reference files#54
migueljette merged 1 commit into
mainfrom
fix/e21-gaap-misspellings

Conversation

@migueljette

Copy link
Copy Markdown
Contributor
  • Corrects 42 tokens across 14 transcripts where gap/Gap/GAP/non-gap were transcribed instead of GAAP/non-GAAP in financial reporting contexts
  • Also fixes the accompanying case tag (LC→CA for GAAP, LC→MC for non-GAAP, UC→CA for Gap) to be consistent with correctly-spelled GAAP tokens elsewhere in the dataset
  ┌─────────┬──────────────┬───────────────────────┐
  │  File   │ Tokens fixed │    Forms corrected    │
  ├─────────┼──────────────┼───────────────────────┤
  │ 4320211 │ 6            │ non-gap×4, gap×2      │
  ├─────────┼──────────────┼───────────────────────┤
  │ 4344866 │ 3            │ non-gap, gap×2        │
  ├─────────┼──────────────┼───────────────────────┤
  │ 4360366 │ 2            │ non-gap, GAP          │
  ├─────────┼──────────────┼───────────────────────┤
  │ 4360674 │ 4            │ non-gap, gap×3        │
  ├─────────┼──────────────┼───────────────────────┤
  │ 4364366 │ 3            │ non-gap×2, gap        │
  ├─────────┼──────────────┼───────────────────────┤
  │ 4365948 │ 2            │ non-gap×2             │
  ├─────────┼──────────────┼───────────────────────┤
  │ 4366302 │ 2            │ non-gap, gap          │
  ├─────────┼──────────────┼───────────────────────┤
  │ 4382825 │ 2            │ non-gap, gap          │
  ├─────────┼──────────────┼───────────────────────┤
  │ 4383161 │ 1            │ gap                   │
  ├─────────┼──────────────┼───────────────────────┤
  │ 4384964 │ 5            │ non-gap, gap×2, Gap×2 │
  ├─────────┼──────────────┼───────────────────────┤
  │ 4387383 │ 3            │ non-gap×2, gap        │
  ├─────────┼──────────────┼───────────────────────┤
  │ 4389907 │ 5            │ gap×5                 │
  ├─────────┼──────────────┼───────────────────────┤
  │ 4397800 │ 2            │ non-gap, gap          │
  ├─────────┼──────────────┼───────────────────────┤
  │ 4397829 │ 2            │ gap×2                 │
  └─────────┴──────────────┴───────────────────────┘

Test plan

  • Verify no remaining gap/Gap/GAP/non-gap tokens in these 14 files that should be GAAP/non-GAAP
  • Confirm the 17 remaining gap tokens in the dataset are legitimate uses (price gaps, strategic gaps, time gaps, etc.) and were intentionally left unchanged

Corrects 42 tokens across 14 transcripts where gap/Gap/GAP/non-gap
were used instead of GAAP/non-GAAP in financial reporting contexts.
Also updates case tags (LC→CA for GAAP, LC→MC for non-GAAP, UC→CA for Gap)
to match correctly spelled tokens elsewhere in the dataset.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@migueljette
migueljette merged commit d5d09d0 into main May 29, 2026
2 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