Skip to content

HR upload field renaming & OM quality check#140

Merged
AlisonLanski merged 8 commits intomasterfrom
dev
Feb 2, 2026
Merged

HR upload field renaming & OM quality check#140
AlisonLanski merged 8 commits intomasterfrom
dev

Conversation

@AlisonLanski
Copy link
Owner

@AlisonLanski AlisonLanski commented Feb 2, 2026

Addresses IPEDS change from names like RACEETHNICITYGENDER to RACEETHNICITYSEX. Also adds a quality check to prep_om that looks for the common error of including multiple degree levels per person per time period, which addresses #139

@AlisonLanski AlisonLanski added this to the Winter/Spring Collection milestone Feb 2, 2026
@AlisonLanski AlisonLanski self-assigned this Feb 2, 2026
@AlisonLanski AlisonLanski added bug Something isn't working enhancement New feature or request labels Feb 2, 2026
@AlisonLanski
Copy link
Owner Author

the automated check fail seems to be based on a server issue for the macos tester. (it can't create a necessary connection), so I'm ok with that for now. Others are working fine as is a manual mac os check for mac_release.

@AlisonLanski AlisonLanski moved this to Ready in Necessary Updates Feb 2, 2026
@AlisonLanski AlisonLanski linked an issue Feb 2, 2026 that may be closed by this pull request
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 90.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 69.90%. Comparing base (cb7a2aa) to head (06a7c9b).

Files with missing lines Patch % Lines
R/prep_om_data_frame.R 66.66% 1 Missing ⚠️
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #140      +/-   ##
==========================================
- Coverage   69.90%   69.90%   -0.01%     
==========================================
  Files          75       75              
  Lines        3064     3067       +3     
==========================================
+ Hits         2142     2144       +2     
- Misses        922      923       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@AlisonLanski AlisonLanski merged commit 75db481 into master Feb 2, 2026
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

Status: Ready

Development

Successfully merging this pull request may close these issues.

Update OM to clarify multi-degree data prep

2 participants