Add liquid asset variables for SSI modeling#7329
Merged
Conversation
MaxGhenis
force-pushed
the
add-liquid-asset-variables
branch
2 times, most recently
from
9f4d22f to
f756a2f
Compare
MaxGhenis
force-pushed
the
add-liquid-asset-variables
branch
from
f756a2f to
193af04
Compare
Contributor
Author
|
Refreshing PR - CI was not running |
Adds three input variables imputed from SIPP in policyengine-us-data: - bank_account_assets: checking, savings, money market - stock_assets: stocks and mutual funds - bond_assets: bonds and government securities Updates ssi_countable_resources to calculate from these asset categories, enabling realistic asset-based eligibility testing for SSI. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Replace formula with adds attribute referencing parameter file that lists the asset sources. This separates policy logic (which assets count) from variable implementation. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…sources The USC only lists exclusions, not what counts. POMS SI 01140.xxx explicitly defines bank accounts, stocks, and bonds as countable. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
spm_unit_cash_assets now sums bank_account_assets, stock_assets, and bond_assets across the SPM unit. This flows through to IL AABD and MA EAEDC which use spm_unit_cash_assets. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Remove microsimulation bypass that used random pass rate. Now that liquid assets are imputed from SIPP, the resource test can use real calculated values. - Remove pass_rate.yaml parameter (no longer needed) - Update README to reflect new methodology - meets_ssi_resource_test now always uses ssi_countable_resources Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add takes_up_ssi_if_eligible with default_value=True - Apply takeup multiplier in ssi.py for microsimulation - Follows new pattern from PR #7326 (no random() in country package) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
MaxGhenis
force-pushed
the
add-liquid-asset-variables
branch
from
da517b5 to
4f8d516
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7329 +/- ##
===========================================
+ Coverage 86.56% 97.77% +11.21%
===========================================
Files 12 8 -4
Lines 134 90 -44
Branches 0 1 +1
===========================================
- Hits 116 88 -28
+ Misses 18 1 -17
- Partials 0 1 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds three input variables for liquid assets (imputed from SIPP in policyengine-us-data):
bank_account_assets: checking, savings, money market accountsstock_assets: stocks and mutual fundsbond_assets: bonds and government securitiesUpdates
ssi_countable_resourcesto calculate from these asset categories.Why separate variables?
Instead of a single
ssi_countable_resourcesinput, we impute the underlying asset categories so:Impact
With the companion policyengine-us-data PR, this enables:
Depends on
Companion data PR: PolicyEngine/policyengine-us-data#511
🤖 Generated with Claude Code