Skip to content

Conversation

@PavelMakarchuk
Copy link
Collaborator

Fixes #4962

@PavelMakarchuk PavelMakarchuk requested review from MaxGhenis and removed request for MaxGhenis August 30, 2024 17:38
@PavelMakarchuk PavelMakarchuk removed the request for review from MaxGhenis September 9, 2024 13:11
@CalebPena
Copy link
Collaborator

@PavelMakarchuk Is this pr still being reviewed?

Copy link
Contributor

@MaxGhenis MaxGhenis left a 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

Copy link
Contributor

Choose a reason for hiding this comment

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

would work right?

Suggested change
adds = ["spm_unit_size"]
subtracts = ["is_snap_ineligible_student"]

Copy link
Collaborator Author

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

@PavelMakarchuk
Copy link
Collaborator Author

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.

Yes as far as I understand students need to meet one of the exemption criteria:

Students attending an institution of higher education half-time or more are only eligible for SNAP if they meet an exemption. These exemptions include meeting any of the following:
Being under age 18 or being 50 or older
Working at least 20 hours a week in paid employment
Participating in a state or federally financed work study program
Having a physical or mental disability

Do we not have an entity build for spm_unit in entities.py

@CalebPena
Copy link
Collaborator

@MaxGhenis @PavelMakarchuk Have we made any progress on this ticket?

@PavelMakarchuk
Copy link
Collaborator Author

@MaxGhenis to answer your question above:
SNAP currently: ~$113 billion.
SNAP after this change: ~$106 billion

@MaxGhenis MaxGhenis merged commit 3177b65 into master Oct 15, 2024
3 checks passed
@MaxGhenis MaxGhenis deleted the PavelMakarchuk/issue4962 branch October 15, 2024 16:32
@MaxGhenis
Copy link
Contributor

I'm still not fully convinced this is the right approach but merging as it seems to solve a pressing need.

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.

Exclude students from the SNAP unit if they are students.

3 participants