Skip to content

Fix GAAP misspellings in NLP reference files#53

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

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

Conversation

@migueljette

Copy link
Copy Markdown
Contributor
  • Corrects 10 tokens across 4 transcripts where gap/Gap/Non-Gap were transcribed instead of GAAP/Non-GAAP in financial reporting contexts
  • Also fixes the accompanying case tag (UC/LC → CA, LC → MC) to be consistent with correctly-spelled GAAP tokens elsewhere in the dataset
  • Changes applied to both nlp_references and force_aligned_nlp_references
┌─────────┬──────────────────────────────┬────────────────────┐
│  File   │         Lines fixed          │       Change       │
├─────────┼──────────────────────────────┼────────────────────┤
│ 4473238 │ 2513, 2547, 2582, 2617, 2639 │ Gap → GAAP         │
├─────────┼──────────────────────────────┼────────────────────┤
│ 4473238 │ 2673                         │ Non-Gap → Non-GAAP │
├─────────┼──────────────────────────────┼────────────────────┤
│ 4475486 │ 2722 / 2912 (aligned)        │ gap → GAAP         │
├─────────┼──────────────────────────────┼────────────────────┤
│ 4481221 │ 246 / 270 (aligned)          │ gap → GAAP         │
├─────────┼──────────────────────────────┼────────────────────┤
│ 4485244 │ 126 / 135 (aligned)          │ non-gap → non-GAAP │
├─────────┼──────────────────────────────┼────────────────────┤
│ 4485244 │ 135 / 144 (aligned)          │ gap → GAAP         │
└─────────┴──────────────────────────────┴────────────────────┘

Test plan

  • Verify no remaining gap/Gap/Non-Gap tokens in these 4 files that should be GAAP
  • Spot-check surrounding context to confirm no unintended changes

Corrects 10 tokens across 4 transcripts where "gap"/"Gap"/"Non-Gap"
were used instead of "GAAP"/"Non-GAAP" in financial reporting contexts.
Also updates the case tag (UC/LC → CA, LC → MC) to match correctly
spelled GAAP tokens elsewhere in the dataset.

Files affected: 4473238, 4475486, 4481221, 4485244
(both nlp_references and force_aligned_nlp_references)

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

@jprobichaud jprobichaud left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@migueljette
migueljette merged commit c05ab6f 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