Repository navigation
feat(cli): support Explore time-series transformations - #2109
Open
nsdeschenes wants to merge 3 commits into
Open
nsdeschenes wants to merge 3 commits into
nsdeschenes wants to merge 3 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
Preserve ordered transformations by request axis position and require returned provenance before exposing transformed values. Keep duplicate aggregates independent and retain nulls, units, and the executed interval. Co-Authored-By: GPT-6.1 Sol <noreply@openai.com>
Add repeatable indexed transformation inputs and show requested transformations with returned provenance. Cover command parsing, SDK invocation, and the real CLI-to-HTTP flow. Regenerate the command reference. Co-Authored-By: GPT-6.1 Sol <noreply@openai.com>
Document ordered per-axis inputs, public SDK usage, provenance, and explicit failures when the response cannot prove transformation execution. Co-Authored-By: GPT-6.1 Sol <noreply@openai.com>
nsdeschenes
force-pushed
the
nd/resolve-exp-1411
branch
from
November 9, 2026 15:12
122053f to
a0d7a9c
Compare
nsdeschenes
marked this pull request as ready for review
November 9, 2026 15:28
This branch was previously deployed
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.
The goal of this PR is to expand transformations support to the CLI for explore time-series requests.
🤖 generated description below:
Why
Fixes EXP-1411. Explore time-series queries need ordered transformations for each requested axis, including independent duplicate aggregates.
What changed
--transform '0:fill(linear)'inputs to the CLI and public CLI SDK.Scope
This covers CLI Explore time-series queries. Dashboard controls and backend transformation execution remain separate work.
Tradeoffs
Empty transformed series fail explicitly because the current backend response cannot prove execution without bucket provenance.
Blast radius
Explicit transformation requests add validation and execution checks. Returned nulls, units, and executed intervals remain intact.
Verification
TZ=UTC pnpm run testremains red on an existing installer test and the Apple asset fixture setup, which lacksactool. The installer failure was also reproduced on untouched HEAD.