Skip to content

Preserve nonfinite intent parameters through plan JSON - #504

Closed
zzylol wants to merge 1 commit into
fix/stored-series-binaryfrom
fix/nonfinite-intent-parameters
Closed

zzylol wants to merge 1 commit into
fix/stored-series-binaryfrom
fix/nonfinite-intent-parameters

Conversation

@zzylol

@zzylol zzylol commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Stacked on #503.

Why

Non-finite scalar literals survived plan JSON, but lowering a literal into an intent parameter still lost it: histogram_quantile(NaN, x_bucket) encoded its quantile as null.

What

Use the shared explicit NaN/Inf encoding for floating intent parameters, including quantiles and prediction horizons. Correct the corpus serialization caveat.

Before this PR

The logical histogram quantile DAG failed JSON decoding with invalid type: null, expected f64.

After this PR

Logical and compiled DAGs preserve the non-finite parameter and return the intended NaN result. Finite numeric encodings stay unchanged.

Validation

Focused logical-DAG regression failed before the fix and passed after. All 41 fallback tests and all-target workspace clippy pass; formatting checked.

🤖 Generated with Claude Code

@zzylol
zzylol force-pushed the feat/handoff-range-bounds branch from 2096e3a to 9ed2e69 Compare September 30, 2026 18:27
@zzylol
zzylol force-pushed the fix/nonfinite-intent-parameters branch from ee17298 to e2985bc Compare September 30, 2026 18:27
@zzylol
zzylol changed the base branch from feat/handoff-range-bounds to fix/stored-series-binary September 30, 2026 18:28
Use the shared float encoding for parameters such as histogram quantiles and prediction horizons; reject old null encodings instead of losing the literal.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@zzylol
zzylol force-pushed the fix/stored-series-binary branch from 9f8c1dc to f914507 Compare September 30, 2026 20:25
@zzylol
zzylol force-pushed the fix/nonfinite-intent-parameters branch from e2985bc to af6a7e6 Compare September 30, 2026 20:25
@zzylol

zzylol commented Oct 1, 2026

Copy link
Copy Markdown
Contributor Author

Included in the squash merge of #507 into main (79f3587).

@zzylol zzylol closed this Oct 1, 2026
@milindsrivastava1997
milindsrivastava1997 deleted the fix/nonfinite-intent-parameters branch November 4, 2026 16:53
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.

1 participant