Skip to content

Accept contained income-range source evidence#63

Merged
MaxGhenis merged 1 commit into
mainfrom
codex/income-range-source-evidence-20260623
Jun 23, 2026
Merged

Accept contained income-range source evidence#63
MaxGhenis merged 1 commit into
mainfrom
codex/income-range-source-evidence-20260623

Conversation

@MaxGhenis

Copy link
Copy Markdown
Contributor

Summary

  • Allow source-row semantic evidence to satisfy a broader income_range filter when the source row range is contained within the aggregate filter
  • Add a focused agent-acceptance regression for 500k_plus facts evidenced by 500k_to_1m and 1m_plus source rows
  • Extend the SOI Historic Table 2 state AGI taxable-interest package test to build the source suite and assert row semantic validation passes

Context

The merged taxable-interest source package can aggregate 500k_plus facts from narrower source rows. Exact string matching caused false row_filter_value_mismatch errors during full bundle builds.

Validation

  • uv run pytest tests/test_arch_suite.py::test_agent_acceptance_accepts_aggregate_income_range_source_rows tests/test_arch_source_package.py::test_soi_historic_table_2_state_agi_package_builds_taxable_interest_facts
  • uv run ruff check arch/suite.py tests/test_arch_suite.py tests/test_arch_source_package.py
  • Local full 2023 bundle build completed valid with 11,455 facts, 0 errors, 1 warning.

@MaxGhenis MaxGhenis merged commit 4638050 into main Jun 23, 2026
2 checks passed
@MaxGhenis MaxGhenis deleted the codex/income-range-source-evidence-20260623 branch June 23, 2026 20:57
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.

1 participant