Skip to content

♻️ polars nullable 입력 지원#20

Merged
mingi3314 merged 1 commit into
mainfrom
polars-nullable-inputs
Apr 27, 2026
Merged

♻️ polars nullable 입력 지원#20
mingi3314 merged 1 commit into
mainfrom
polars-nullable-inputs

Conversation

@mingi3314
Copy link
Copy Markdown
Collaborator

요약

  • 이 PR은 techr-core의 nullable 입력 계약에 맞춰 Polars plugin 입력 변환을 Vec<Option<f64>> 기반으로 정렬합니다.
  • 기존에는 null 입력이 있으면 Polars ComputeError를 반환했지만, 이제 core의 rolling-window/seed recovery semantics에 따라 row-aligned null 결과를 반환합니다.
  • #15에서 core-only 변경을 통과시키기 위해 넣었던 Polars CI no-op preflight를 제거하고, core/** 변경에서도 Polars CI가 다시 실제 검증을 수행하도록 복원합니다.
  • Polars 패키지 버전을 0.1.1에서 0.1.2로 bump합니다.

리뷰 포인트

  • polars/src/expressions.rsSeries -> Vec<Option<f64>> 변환이 기존 numeric cast 동작을 유지하면서 null bitmap을 보존하는지 확인하면 됩니다.
  • nullable 입력 테스트는 shared conversion path를 single-input/multi-input 및 eager/lazy 양쪽에서 대표 검증합니다.
  • .github/workflows/polars-ci.yml♻️ core 지표 nullable 입력 계약 도입 #15 직전 형태로 되돌리는 변경입니다.

테스트

  • cargo check -p polars_techr
  • cargo fmt --check
  • cd polars && uv run ruff check .
  • cargo test -p techr-core
  • cd polars && uv run maturin develop --uv
  • cd polars && uv run pytest
  • cd polars && uv run maturin build --release --sdist --out dist
  • cd polars && uv run python scripts/check_artifacts.py dist
  • cd polars && uv run --isolated --python 3.10 --with <built-wheel> python scripts/smoke_import.py

참고

  • 로컬 maturin develop에서는 patchelf 미설치로 rpath 경고가 출력됐지만, editable 설치와 테스트는 정상 완료됐습니다.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

#15 에서 CI 임시 비활성화를 위해 조치해둔 수정사항을 원복시켜뒀습니다

@alphaprime-dev-discord
Copy link
Copy Markdown

@mingi3314 mingi3314 requested a review from sjquant April 27, 2026 05:01
@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.

👍 👍 👍

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