Skip to content

✨ TRIX, TRIX signal 지표 추가#23

Merged
mingi3314 merged 4 commits into
mainfrom
feat/trix-indicator
May 13, 2026
Merged

✨ TRIX, TRIX signal 지표 추가#23
mingi3314 merged 4 commits into
mainfrom
feat/trix-indicator

Conversation

@mingi3314
Copy link
Copy Markdown
Collaborator

변경 사항

  • core에 TRIX line과 TRIX signal 계산 함수를 추가했습니다.
  • triple EMA와 signal EMA를 입력 순회 중 상태로 계산해 불필요한 중간 벡터 생성을 줄였습니다.
  • Polars plugin과 Python wrapper에 trix, trix_line, trix_signal API를 노출했습니다.
  • TSLA, 005930 기준 fixture와 Polars indicator 매트릭스 테스트를 추가했습니다.
  • polars 패키지 patch 버전을 0.1.3에서 0.1.4로 올렸습니다.

검증

  • cd core && cargo test
  • cd polars && uv run pytest

@mingi3314 mingi3314 changed the title ✨ TRIX 지표 추가와 패치 버전 갱신 ✨ TRIX, TRIX signal 지표 추가 May 13, 2026
@mingi3314 mingi3314 requested a review from sjquant May 13, 2026 05:34
@mingi3314 mingi3314 self-assigned this May 13, 2026
@alphaprime-dev-discord
Copy link
Copy Markdown

Comment thread core/src/indicators/trix.rs Outdated
Comment thread core/src/indicators/trix.rs
Comment thread polars/tests/test_indicators.py Outdated
@mingi3314
Copy link
Copy Markdown
Collaborator Author

리뷰 코멘트 반영했습니다.

  • TRIX 계산에서 별도 EmaState struct를 제거하고 기존 ema()를 합성하는 방식으로 변경했습니다.
  • TRIX 관련 core 테스트에 Given / When / Then 블록을 추가했습니다.
  • Polars 레벨에서 중복 API였던 trix alias를 제거하고 trix_line, trix_signal만 남겼습니다.

검증:

  • cd core && cargo test 통과
  • cd polars && uv run pytest 통과

@mingi3314 mingi3314 force-pushed the feat/trix-indicator branch from 09cc9eb to 4f1214d Compare May 13, 2026 06:30
@mingi3314 mingi3314 requested a review from sjquant May 13, 2026 06:32
Copy link
Copy Markdown
Collaborator

@sjquant sjquant left a comment

Choose a reason for hiding this comment

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

👍 👍 👍

@mingi3314 mingi3314 merged commit 8b4fc14 into main May 13, 2026
1 check 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