♻️ polars nullable 입력 지원#20
Merged
Merged
Conversation
mingi3314
commented
Apr 27, 2026
Collaborator
Author
There was a problem hiding this comment.
#15 에서 CI 임시 비활성화를 위해 조치해둔 수정사항을 원복시켜뒀습니다
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
요약
techr-core의 nullable 입력 계약에 맞춰 Polars plugin 입력 변환을Vec<Option<f64>>기반으로 정렬합니다.ComputeError를 반환했지만, 이제 core의 rolling-window/seed recovery semantics에 따라 row-aligned null 결과를 반환합니다.core/**변경에서도 Polars CI가 다시 실제 검증을 수행하도록 복원합니다.0.1.1에서0.1.2로 bump합니다.리뷰 포인트
polars/src/expressions.rs의Series -> Vec<Option<f64>>변환이 기존 numeric cast 동작을 유지하면서 null bitmap을 보존하는지 확인하면 됩니다..github/workflows/polars-ci.yml은 ♻️ core 지표 nullable 입력 계약 도입 #15 직전 형태로 되돌리는 변경입니다.테스트
cargo check -p polars_techrcargo fmt --checkcd polars && uv run ruff check .cargo test -p techr-corecd polars && uv run maturin develop --uvcd polars && uv run pytestcd polars && uv run maturin build --release --sdist --out distcd polars && uv run python scripts/check_artifacts.py distcd polars && uv run --isolated --python 3.10 --with <built-wheel> python scripts/smoke_import.py참고
maturin develop에서는patchelf미설치로 rpath 경고가 출력됐지만, editable 설치와 테스트는 정상 완료됐습니다.