Skip to content

fix: reports 지표 순서 호환성 복원#8

Merged
mingi3314 merged 1 commit into
mainfrom
fix/legacy-report-metrics-order
May 13, 2026
Merged

fix: reports 지표 순서 호환성 복원#8
mingi3314 merged 1 commit into
mainfrom
fix/legacy-report-metrics-order

Conversation

@mingi3314

Copy link
Copy Markdown
Collaborator

리뷰어가 작업내용을 이해하기 쉽도록 아래내용을 상세히 작성해주세요.

Changes

  • reports.metrics()의 basic/full report 항목 집합과 순서를 QuantStats legacy report와 호환되도록 조정했습니다.
  • 클라이언트가 reports 배열의 항목과 순서에 의존하는 케이스를 위해, Max DD Date, Max DD Period Start, Max DD Period End, Avg. Return, Avg. Win, Avg. Loss row를 제거하고 Expected Daily, Expected Monthly, Expected Yearly row를 복원했습니다.
  • legacy basic/full report metric order를 테스트로 고정했습니다.
  • patch 버전을 0.2.1로 올렸습니다.

Test

💡 진행한 테스트에 대해서 구체적으로 작성해주세요.

  • uv run pytest -q
  • uv run ruff check alphastats/reports.py tests/test_quantstats_parity.py
  • uv run mypy alphastats tests/test_quantstats_parity.py
  • QuantStats 0.0.62와 basic/full reports row 개수 및 순서가 일치하는 것을 로컬 스크립트로 확인했습니다.

Task Link

💡 관련 페이지 링크를 작성해주세요.

  • N/A

@mingi3314
mingi3314 requested a review from chado13 May 13, 2026 01:13
@mingi3314 mingi3314 self-assigned this May 13, 2026
@alphaprime-dev-discord

Copy link
Copy Markdown

@mingi3314
mingi3314 merged commit 7522a8a into main May 13, 2026
2 checks passed
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.

2 participants