Skip to content

Remove aggregation method from Ledger target selectors#193

Merged
MaxGhenis merged 1 commit into
mainfrom
codex/remove-aggregation-selector-20260626
Jun 26, 2026
Merged

Remove aggregation method from Ledger target selectors#193
MaxGhenis merged 1 commit into
mainfrom
codex/remove-aggregation-selector-20260626

Conversation

@MaxGhenis

Copy link
Copy Markdown
Contributor

Summary

  • remove aggregation_method as a selectable Ledger target selector field
  • keep Ledger aggregation only as source lineage metadata and sum-only validation
  • update tests so count aggregation remains a rejected input, not a target option

Tests

  • uv run --project packages/populace-build --group dev python -m pytest packages/populace-build/tests/test_ledger_targets.py packages/populace-calibrate/tests/test_target.py -q
  • uv run ruff check packages/populace-build/src/populace/build/ledger_targets.py packages/populace-build/tests/test_ledger_targets.py packages/populace-build/tests/test_us_fiscal_targets.py
  • git diff --check

@MaxGhenis MaxGhenis merged commit 23cc25d into main Jun 26, 2026
4 checks passed
@MaxGhenis MaxGhenis deleted the codex/remove-aggregation-selector-20260626 branch June 26, 2026 21:23
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