Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update variable name for OG-Core 0.11.13 #120

Merged
merged 1 commit into from
Jul 29, 2024

Conversation

jdebacker
Copy link
Member

This PR changes the name of the variable AIME_num_years to avg_earn_num_years, as was introduced in OG-Core version 0.11.13.

@jdebacker
Copy link
Member Author

Resolves Issue #119

@codecov-commenter
Copy link

codecov-commenter commented Jul 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 45.12%. Comparing base (f20a9fe) to head (15afb95).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #120   +/-   ##
=======================================
  Coverage   45.12%   45.12%           
=======================================
  Files          18       18           
  Lines        1405     1405           
=======================================
  Hits          634      634           
  Misses        771      771           
Flag Coverage Δ
unittests 45.12% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@jdebacker jdebacker merged commit 5e93add into PSLmodels:master Jul 29, 2024
7 checks passed
@jdebacker jdebacker deleted the var_name branch July 29, 2024 00:18
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