♻️ core 복합 지표 nullable 입력 전환 by mingi3314 · Pull Request #17 · alphaprime-dev/techr

mingi3314 · 2026-04-21T08:09:56Z

요약

이 PR은 #16 위에 쌓인 stacked PR입니다. 리뷰는 core-nullable-stateful 대비로 봐주시면 됩니다.
복합/파생 indicator를 nullable contract에 맞추고, techr-core 전반의 public API 정합성을 마무리합니다.
하위 helper/stateful layer에서 정의된 gap semantics가 상위 지표에서도 그대로 보존되도록 정리합니다.

변경 사항

macd, ppo, pvo, massi, sonar, stochf, stochs, stochrsi 등 composite indicator를 nullable 입력 기준으로 전환합니다.
aroon, aroonosc, cci, cv, ichimoku, pchan, willr, roc, mom, psl, erbull, erbear 등 남아 있던 파생 indicator를 같은 계약으로 맞춥니다.
내부 helper 사용 경로에서 dense-only 가정이나 NaN 기반 우회가 남아 있던 부분을 정리합니다.
길이 불일치와 gap 전파가 섞인 경계 케이스를 보강합니다.

리뷰 포인트

composite indicator가 하위 indicator의 gap semantics를 그대로 전파하는지
rolling extrema 계열이 gap이 들어간 window를 valid 값으로 오판하지 않는지
signal/oscillator 계열이 nullable intermediate를 압축하지 않고 aligned output을 유지하는지
stoch*, macd/ppo/pvo, ichimoku, aroon 계열이 대표적인 확인 포인트입니다.

범위 외

plugin/polars nullable adapter 마이그레이션은 후속 작업입니다.

테스트 계획

cargo fmt --package techr-core --check
cargo test -p techr-core

mingi3314 · 2026-04-21T08:10:10Z

This stack of pull requests is managed by Graphite. Learn more about stacking.

alphaprime-dev-discord · 2026-04-23T08:58:33Z

디스코드 스레드 바로가기

mingi3314 · 2026-04-24T02:31:47Z

후속 cleanup만 공유드립니다. approve 이후 최신 head(801ccdc)에서 thin alias였던 ema_aligned / sma_aligned를 제거하고, 남아 있던 forwarding-only rolling wrapper들도 public 함수로 인라인했습니다. 동작 의미는 바꾸지 않았고, cargo fmt --package techr-core --check, cargo test -p techr-core 재검증까지 완료했습니다.

mingi3314 · 2026-04-27T02:16:06Z

Merge activity

Apr 27, 2:16 AM UTC: A user started a stack merge that includes this pull request via Graphite.
Apr 27, 2:19 AM UTC: Graphite rebased this pull request as part of a merge.
Apr 27, 2:20 AM UTC: @mingi3314 merged this pull request with Graphite.

## 요약 - 이 PR은 `techr-core`의 nullable 입력 마이그레이션을 시작하는 bottom PR입니다. - 공개 indicator API의 결측치 계약을 `NaN`이 아닌 `&[Option<f64>]` aligned series로 정리하고, 이후 stacked PR인 `#16`, `#17`이 이 규칙 위에서 stateful/composite 지표를 순차적으로 올립니다. - 범위는 foundation helper와 단일 시계열 기반 지표 중심이며, `plugin/polars`의 실제 nullable adapter 마이그레이션은 이번 스택에서 제외합니다. ## 배경 - 기존 `core` 함수들은 대부분 dense `&[f64]` 입력을 전제하고 있었습니다. - 하지만 실제 시계열 처리에서는 결측이 섞인 aligned series를 직접 받을 수 있어야 하고, 표준 라이브러리 성격의 API라면 missing을 `NaN`이 아니라 명시적인 타입으로 다루는 편이 계약이 더 분명합니다. - 이 스택의 목표는 `techr-core` 전체에서 nullable aligned input을 표준 계약으로 만들고, gap을 건너뛰며 series를 압축하지 않는 일관된 의미론을 정착시키는 것입니다. ## 이 PR에서 하는 일 - `core/src/utils.rs`에 nullable rolling/helper primitive를 추가해 foundation semantics를 먼저 고정합니다. - `sma`, `ema`, `wma`, `bband`, `env`, `disparity`를 `&[Option<f64>]` 입력 기준으로 전환합니다. - foundation helper를 사용하는 일부 downstream indicator의 public 시그니처도 같은 계약에 맞게 정렬합니다 (`macd`, `ppo`, `pvo`, `massi`, `sonar` 등). - 공개 API의 결측치 표현으로 `NaN`을 새로 도입하지 않도록 정리합니다. ## 핵심 규칙 - 출력 길이는 입력 정렬을 그대로 유지합니다. - rolling 계산은 필요한 lookback window 안에 gap이 있으면 해당 row를 `None`으로 반환합니다. - recursive smoothing은 gap row에서 output만 `None`이고 내부 상태는 유지합니다. - valid 값만 따로 압축해서 계산한 뒤 다시 원래 위치에 펴는 방식을 public contract로 사용하지 않습니다. ## 범위 외 - `plugin/polars` nullable adapter 마이그레이션은 후속 작업으로 미룹니다. - 다만 현재 CI에서 `Polars CI`가 core-only PR에도 실행되기 때문에, 이 PR에는 non-polars 변경에서 `validate`를 성공적인 no-op으로 처리하도록 하는 최소한의 workflow 조정이 함께 포함됩니다. - 이 workflow 변경은 core nullable 전환 자체의 기능 스코프가 아니라, deferred plugin migration이 현재 스택을 막지 않도록 하는 CI 보정입니다. ## 리뷰 가이드 - 먼저 `core/src/utils.rs`, `sma.rs`, `ema.rs`, `wma.rs`, `bband.rs`, `disparity.rs`를 봐주시면 됩니다. - 그 다음 downstream 시그니처 정렬과 `.github/workflows/polars-ci.yml` 변경을 확인하시면 충분합니다. - 이 PR만 `main` 기준으로 리뷰하면 되고, 상위 PR인 `#16`, `#17`은 각각 이 PR을 base로 봐주시면 됩니다. ## 테스트 계획 - [x] `cargo fmt --package techr-core --check` - [x] `cargo test -p techr-core` - [x] GitHub Actions `Polars CI`가 non-polars 변경에서 성공적으로 통과하는지 확인

## 요약 - 이 PR은 `#15` 위에 쌓인 stacked PR입니다. 리뷰는 `core-nullable-foundations` 대비로 봐주시면 됩니다. - foundation layer에서 정의한 nullable contract를 상태형, pairwise, cumulative 지표에 적용합니다. - gap을 건너뛰며 series를 압축하지 않고, 현재 row, 이전 row, 내부 state의 유효성에 따라 `None`을 반환하는 규칙을 맞춥니다. ## 변경 사항 - `rsi`, `atr`, `dmi`, `adx`, `adxr`를 nullable 입력 기준으로 전환합니다. - `ad`, `cmf`, `co`, `efi`, `eom`, `nvi`, `obv`, `pvi`, `ultosc`, `vr` 등 pairwise/cumulative family를 같은 계약으로 정리합니다. - `psar`를 포함한 stateful indicator가 gap row에서 output만 `None`을 내고, 다음 valid row에서 이전 state를 이어서 재개하도록 맞춥니다. - 필요한 nullable helper를 `core/src/utils.rs`에 추가합니다. ## 리뷰 포인트 - current row나 required predecessor가 비면 해당 row는 `None`이어야 합니다. - recursive/stateful 계산은 gap을 사이에 두고 series를 압축하지 않은 채 정렬을 유지해야 합니다. - accumulator 기반 지표는 invalid row에서 상태를 억지로 갱신하지 않아야 합니다. - `psar`, `dmi`, `atr`, `rsi`가 대표적인 확인 포인트입니다. ## 범위 외 - `plugin/polars`는 여전히 이번 stack 범위 밖입니다. - composite/derived indicator의 최종 정리는 상위 PR `#17`에서 다룹니다. ## 테스트 계획 - [x] `cargo fmt --package techr-core --check` - [x] `cargo test -p techr-core`

## 요약 - 이 PR은 `#17` 위에 쌓인 stacked PR입니다. 리뷰는 `core-nullable-composites` 대비로 봐주시면 됩니다. - 남아 있던 dense-to-nullable bridge helper인 `ema_dense`, `rsi_dense`를 제거하고, 마지막 adapter 의존 indicator를 직접 nullable 입력 계약으로 전환합니다. - 하위 PR들에서 이미 정리한 nullable semantics를 그대로 재사용하면서, dense adapter 층만 걷어내는 좁은 범위의 cleanup입니다. ## 변경 사항 - `erbull`, `erbear`, `cv`, `stochrsi`를 `&[Option<f64>]` 입력 기준으로 전환합니다. - 각 indicator가 `ema`, `rsi` 같은 nullable 본체를 직접 사용하도록 정리합니다. - `ema_dense`, `rsi_dense`를 제거합니다. - gap 전파와 재개 동작이 기존 nullable contract를 그대로 따르는지 확인하는 테스트를 보강합니다. ## 리뷰 포인트 - touched indicator가 dense adapter 없이도 동일한 aligned nullable semantics를 유지하는지 - `EMA`/`RSI` 기반 파생 지표가 gap 이후 조기 emission 없이 기존 contract를 그대로 따르는지 - `core` 내부에 `_dense` adapter가 더 이상 남지 않는지 ## 범위 외 - 남아 있던 dense public indicator API의 최종 nullable 전환은 상위 PR `#19`에서 다룹니다. - `plugin/polars` nullable adapter 마이그레이션은 이번 stack 범위 밖입니다. ## 테스트 계획 - [x] `cargo fmt --package techr-core --check` - [x] `cargo test -p techr-core`

This was referenced Apr 21, 2026

♻️ core 상태형 지표 nullable 입력 전환 #16

Merged

♻️ core 지표 nullable 입력 계약 도입 #15

Merged

mingi3314 force-pushed the core-nullable-composites branch from b394c31 to c054abd Compare April 23, 2026 00:57

mingi3314 force-pushed the core-nullable-stateful branch from f327ee2 to f266b37 Compare April 23, 2026 00:57

mingi3314 changed the base branch from core-nullable-stateful to graphite-base/17 April 23, 2026 01:47

mingi3314 force-pushed the core-nullable-composites branch from c054abd to f0c897c Compare April 23, 2026 01:47

mingi3314 force-pushed the graphite-base/17 branch from f266b37 to 3bbbf2f Compare April 23, 2026 01:47

mingi3314 changed the base branch from graphite-base/17 to core-nullable-stateful April 23, 2026 01:47

mingi3314 changed the title ~~Support nullable core composite indicators~~ ♻️ core 복합 지표 nullable 입력 전환 Apr 23, 2026

mingi3314 force-pushed the core-nullable-composites branch 3 times, most recently from aaafe57 to 37d683c Compare April 23, 2026 08:20

mingi3314 requested a review from sjquant April 23, 2026 08:57

mingi3314 self-assigned this Apr 23, 2026

sjquant approved these changes Apr 24, 2026

View reviewed changes

mingi3314 force-pushed the core-nullable-composites branch from 37d683c to b017452 Compare April 24, 2026 01:43

mingi3314 force-pushed the core-nullable-stateful branch from c84e6f4 to 307a195 Compare April 24, 2026 01:43

mingi3314 force-pushed the core-nullable-composites branch 2 times, most recently from 21c77cd to db48265 Compare April 24, 2026 04:38

mingi3314 force-pushed the core-nullable-stateful branch from b00b2b2 to 1afb218 Compare April 24, 2026 04:38

This was referenced Apr 24, 2026

♻️ core dense adapter 제거 #18

Merged

♻️ core 잔여 지표 nullable 입력 전환 #19

Merged

mingi3314 changed the base branch from core-nullable-stateful to graphite-base/17 April 27, 2026 02:17

mingi3314 changed the base branch from graphite-base/17 to main April 27, 2026 02:18

mingi3314 added 6 commits April 27, 2026 02:19

Support nullable core composite indicators

db4d0b5

Reseed sparse composite signals after gaps

656439e

Trim composite lane to sparse reseeds

f86f146

Align composite signals with nullable EMA contract

c13ad09

Remove thin EMA and SMA aliases

f9167c3

Fix dense deque helpers after restack

bcd6007

mingi3314 force-pushed the core-nullable-composites branch from e58c5f9 to bcd6007 Compare April 27, 2026 02:19

mingi3314 merged commit 85b75a4 into main Apr 27, 2026
1 check passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

♻️ core 복합 지표 nullable 입력 전환#17

♻️ core 복합 지표 nullable 입력 전환#17
mingi3314 merged 6 commits into
mainfrom
core-nullable-composites

mingi3314 commented Apr 21, 2026 •

edited

Loading

Uh oh!

mingi3314 commented Apr 21, 2026 •

edited

Loading

Uh oh!

alphaprime-dev-discord Bot commented Apr 23, 2026

Uh oh!

mingi3314 commented Apr 24, 2026

Uh oh!

mingi3314 commented Apr 27, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

mingi3314 commented Apr 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

요약

변경 사항

리뷰 포인트

범위 외

테스트 계획

Uh oh!

mingi3314 commented Apr 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

alphaprime-dev-discord Bot commented Apr 23, 2026

Uh oh!

mingi3314 commented Apr 24, 2026

Uh oh!

mingi3314 commented Apr 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge activity

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

mingi3314 commented Apr 21, 2026 •

edited

Loading

mingi3314 commented Apr 21, 2026 •

edited

Loading

mingi3314 commented Apr 27, 2026 •

edited

Loading