fix: derive all aggregation timeframes from trade timestamp (#112) #113

agentson · 2026-02-10T00:41:32+09:00

agentson commented

2026-02-10 00:41:32 +09:00

Summary

run_all_aggregations()에서 모든 레이어(L6~L2)의 timeframe을 최신 trade timestamp 기준으로 일관되게 파생하도록 수정
datetime.fromisoformat()의 "Z" suffix 호환성 처리 추가
테스트에서 L4(monthly), L3(quarterly), L2(annual) 레이어까지 정확한 값 검증으로 강화

Closes #112

Test plan

pytest -v --cov=src 전체 488 tests passed
test_run_all_aggregations에서 L6~L2 모든 레이어 end-to-end 값 검증 통과
trade 날짜와 현재 날짜가 다른 주/월에 속해도 정상 집계 확인

🤖 Generated with Claude Code

## Summary - `run_all_aggregations()`에서 모든 레이어(L6~L2)의 timeframe을 최신 trade timestamp 기준으로 일관되게 파생하도록 수정 - `datetime.fromisoformat()`의 `"Z"` suffix 호환성 처리 추가 - 테스트에서 L4(monthly), L3(quarterly), L2(annual) 레이어까지 정확한 값 검증으로 강화 Closes #112 ## Test plan - [x] `pytest -v --cov=src` 전체 488 tests passed - [x] `test_run_all_aggregations`에서 L6~L2 모든 레이어 end-to-end 값 검증 통과 - [x] trade 날짜와 현재 날짜가 다른 주/월에 속해도 정상 집계 확인 🤖 Generated with [Claude Code](https://claude.com/claude-code)

agentson added 1 commit 2026-02-10 00:41:32 +09:00

fix: derive all aggregation timeframes from trade timestamp (#112 )

CI / test (pull_request) Has been cancelled

Details

f03cc6039b

run_all_aggregations() previously used datetime.now(UTC) for weekly
through annual layers while using the trade date only for daily,
causing data misalignment on backfill. Now all layers consistently
use the latest trade timestamp. Also adds "Z" suffix handling for
fromisoformat() compatibility and strengthens test assertions to
verify L4-L2 layer values end-to-end.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

jihoson merged commit c5831966ed into main

2026-02-10 00:42:39 +09:00

jihoson referenced this issue from a commit

2026-02-10 00:42:40 +09:00

Merge pull request 'fix: derive all aggregation timeframes from trade timestamp (#112)' (#113) from fix/test-failures into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: jihoson/The-Ouroboros#113