-
Notifications
You must be signed in to change notification settings - Fork 201
Add child age requirements for SNAP student ineligibility #6735
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
Draft
hua7450
wants to merge
13
commits into
PolicyEngine:master
Choose a base branch
from
hua7450:hua7450/issue6731
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
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
…lopment - Remove .claude submodule configuration - Add .claude directory contents as tracked files - Remove .gitmodules as .claude was the only submodule - Enable direct editing of agent prompts for experimentation This allows for easier iteration on agent prompt improvements without managing a separate submodule repository.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #6735 +/- ##
============================================
+ Coverage 74.69% 100.00% +25.30%
============================================
Files 3249 7 -3242
Lines 46803 85 -46718
Branches 242 0 -242
============================================
- Hits 34959 85 -34874
+ Misses 11807 0 -11807
+ Partials 37 0 -37
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:
|
PavelMakarchuk
requested changes
Oct 27, 2025
policyengine_us/parameters/gov/usda/snap/student/young_child.yaml
Outdated
Show resolved
Hide resolved
policyengine_us/parameters/gov/usda/snap/student/young_child.yaml
Outdated
Show resolved
Hide resolved
policyengine_us/parameters/gov/usda/snap/student/younger_child.yaml
Outdated
Show resolved
Hide resolved
policyengine_us/variables/gov/usda/snap/eligibility/student/is_snap_ineligible_student.py
Show resolved
Hide resolved
Implements Federal Work Study program integration with SNAP student eligibility and income exclusion rules: Student Eligibility (7 USC 2015(e)): - Add is_federal_work_study_participant variable - Add is_part_time_college_student variable - Refactor is_snap_ineligible_student to check both full-time and part-time students (statute requires "at least half-time") - Add Federal Work Study as Exception 4 (work requirement exception) - Create modular exception variables: is_snap_higher_ed_student, meets_snap_work_exception, meets_snap_parent_exception - Document all 8 statutory exceptions with proper section references Income Exclusion (7 CFR 273.9(c)(3)): - Update snap_countable_earner to exclude Federal Work Study participants' income from SNAP calculations - Add reference to 7 CFR 273.9(c)(3) for work-study income exclusion Testing: - Add unit tests for all new variables - Add integration tests verifying income exclusion - Update snap_countable_earner tests for work-study cases 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
Corrected parameter descriptions to accurately reflect Exception 5 and 8: - young_child (6): For two-parent households (Exception 5) - younger_child (12): For single parents (Exception 8) The formula correctly uses the more lenient age threshold (12) for single parents who can qualify under either Exception 5 or 8.
Resolved conflicts in: - is_snap_ineligible_student.yaml: Use weekly_hours_worked_before_lsr - is_snap_ineligible_student.py: Keep new detailed 8-exception formula - meets_snap_work_exception.py: Update to use weekly_hours_worked_before_lsr All Federal Work Study functionality preserved with updated variable names.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Fixes #6731