Repository navigation
Conversation
hua7450
marked this pull request as draft
October 8, 2026 13:38
This branch has not been deployed
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
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.
What and why
Rest repeats full-model setup and runs policy examples that the YAML suites can cover. Reuse the loaded baseline in seven read-only consumers, accelerate repeated YAML scans with compiled safe/base loaders (keeping the Python fallback), and move ordinary policy examples out of Python.
core/test_ssi_disability_criteria_inputs.py: both checks are already covered bygov/ssa/ssi/meets_ssi_disability_criteria.yaml.AGENTS.mdandCLAUDE.md.The migration removes 17 Python cases: 14 moved to YAML and three duplicates deleted, including one entire Python file. Together with the earlier runner/syntax duplicates, this PR removes five duplicate checks. January-only Medicare eligibility inputs remain separate from existing annual-input YAML cases because their calculation paths differ.
The workflow still uses 26 runners, with sequential process isolation. Medicare cases use the existing Baseline
contrib-hhsjob; farm-income cases use Baselineirs; retained SSI coverage uses Baselinessa-usda. No workflow, formula, parameter, or partner test changes, and no new YAML reform combinations.Whole-suite measurements
The final CI run at
25ac672f8cpassed all 26 jobs using 26 runner allocations. Rest remains the last job to finish.Compare the #9744 baseline, including #10006 and #10007 and the earlier setup/scan revision:
5f3ce4573525ac672f8cThe final full Rest job was 3m14s shorter than the original baseline, with highest process RSS falling from 14,242,068 to 11,113,604 KiB. However, total runner job time was 18m48s higher than the original baseline. Compared with the earlier setup/scan revision, final Rest was 10m16s longer and total runner job time was 50m54s higher. This run therefore does not demonstrate additional savings from the deletion/YAML migration, or an overall reduction in CI runner time. Moving work between jobs alone is not evidence of lower total cost.
These are observational comparisons across distinct runner allocations. Baseline and final Rest used Python 3.14.7 and Ubuntu image 20260927.320.1; the earlier setup/scan run used Python 3.14.8 and image 20261004.327.1. Rest installed Core 3.32.21 in all three runs. The PR base also advanced from the #9744-era baseline through #9745; the setup/scan and final revisions share that base. Unchanged test groups varied materially, so neither improvements nor regressions here can be attributed solely to this patch.
Workflow elapsed is measured from the workflow start to the last job completion, including queue/dependency waits. Aggregate runner time sums each allocated job's start-to-completion duration, excluding queue waits; it is not a billing-minutes figure. RSS is the maximum reported by a single timed Rest process, not a measurement of the whole machine or concurrent jobs.
Final Rest artifacts contain 3,273 cases: 3,262 passed and 11 skipped, with zero failures/errors. They comprise 3,204 Python cases and 69 variables YAML cases. The 17-case reduction from the setup/scan run matches the 14 migrated and three deleted Python cases. Seven complete GNU time/memory reports, seven JUnit reports, full Rest logs, and every job/step's metadata have been saved for audit.
Validation
make format,git diff --check, and changelog fragment.25ac672f8c: 26/26 CI jobs passed; full Rest timing/memory, affected YAML jobs, workflow elapsed, and aggregate runner time captured above.axiom: n/a: test setup, deduplication, migration, and documentation only; no policy changes.