-
Notifications
You must be signed in to change notification settings - Fork 182
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
Exclude students from the SNAP unit with certain exceptions. #4975
Conversation
@PavelMakarchuk Is this pr still being reviewed? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you also clarify how this works -- are students just not in any SNAP unit if they don't meet these criteria? They can't get SNAP as their own unit either?
Do we need to make a new entity for SNAP units?
Please also check the total SNAP expenditure and participation before and after this change.
entity = SPMUnit | ||
label = "SNAP unit" | ||
definition_period = YEAR | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would work right?
adds = ["spm_unit_size"] | |
subtracts = ["is_snap_ineligible_student"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I capped the formula at 0 to avoid negative values for units with just one student
policyengine_us/parameters/gov/usda/snap/student/age_threshold.yaml
Outdated
Show resolved
Hide resolved
policyengine_us/parameters/gov/usda/snap/student/working_hours_threshold.yaml
Outdated
Show resolved
Hide resolved
Yes as far as I understand students need to meet one of the exemption criteria:
Do we not have an entity build for spm_unit in entities.py |
@MaxGhenis @PavelMakarchuk Have we made any progress on this ticket? |
@MaxGhenis to answer your question above: |
I'm still not fully convinced this is the right approach but merging as it seems to solve a pressing need. |
Fixes #4962