Skip to content

✨ polars 전체 지표 지원#21

Merged
mingi3314 merged 1 commit intomainfrom
polars-all-indicators
Apr 28, 2026
Merged

✨ polars 전체 지표 지원#21
mingi3314 merged 1 commit intomainfrom
polars-all-indicators

Conversation

@mingi3314
Copy link
Copy Markdown
Collaborator

@mingi3314 mingi3314 commented Apr 27, 2026

변경 사항

  • Polars plugin에서 core의 나머지 기술적 지표 출력을 개별 pl.Expr 함수로 노출했습니다.
  • tuple 기반 core 지표는 *_line, *_signal, *_upper, *_lower, dmi_plus, dmi_minus, stochrsi_percent_k/d처럼 단일 출력 API로 분리했습니다.
  • Python wrapper의 __all__과 README 지원 지표 목록을 갱신했습니다.
  • 모든 신규 Polars API를 core expected fixture와 eager/lazy 양쪽에서 비교하도록 테스트 매트릭스를 확장했습니다.

검토 포인트

  • 다중 출력 지표를 구조체 반환이 아닌 개별 expression 함수로 노출하도록 방향성을 잡았습니다
  • macd_hist는 기존 호환성을 위해 유지하고, fixture 이름과 맞는 macd_histogram도 추가했습니다.
  • core fixture가 제한된 정밀도로 검증하던 ad, co, stochrsi 계열은 Polars 테스트에서도 같은 수준의 허용 오차를 적용했습니다.

검증

  • uv run ruff format techr tests
  • uv run ruff check techr tests
  • make test-polars (294 passed)

@mingi3314 mingi3314 requested a review from sjquant April 27, 2026 05:23
@mingi3314 mingi3314 self-assigned this Apr 27, 2026
@alphaprime-dev-discord
Copy link
Copy Markdown

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.

GREAT!!! 👍

@mingi3314 mingi3314 merged commit f33fbb0 into main Apr 28, 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