Skip to content

Accept calendar-month label horizons - #63

Merged
stefan-jansen merged 2 commits into
mainfrom
codex/monthly-label-horizon
Sep 9, 2026
Merged

Accept calendar-month label horizons#63
stefan-jansen merged 2 commits into
mainfrom
codex/monthly-label-horizon

Conversation

@stefan-jansen

Copy link
Copy Markdown
Contributor

Closes #60

Summary

  • accept 1M, P1M, and integer multiples as label horizons
  • normalize each month to the documented fixed 30-day duration required by purge arithmetic
  • improve invalid-duration guidance and document the approximation

Verification

  • reproduced the prior WalkForwardConfig(label_horizon="1M") validation failure
  • uv run pytest tests/test_splitters/test_config.py tests/test_splitters/test_walk_forward.py tests/test_splitters/test_calendar_first.py -q -n 0 (83 passed)
  • uv run ty check
  • pre-commit run --all-files

Copilot AI lite review requested due to automatic review settings September 9, 2026 05:12

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@stefan-jansen
stefan-jansen merged commit f777219 into main Sep 9, 2026
36 checks passed
@stefan-jansen
stefan-jansen deleted the codex/monthly-label-horizon branch September 9, 2026 17:32
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.

Support calendar-month label horizons in WalkForwardConfig

2 participants