Conversation
…al, Wren's measurements)
|
Recreating this fresh at AmirHossein's request — same |
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.
Standing PR for board changes. Board-only (
backlog/**) — never code.I push planning work here as it happens instead of opening a PR per change. Merge it whenever suits you; there's no urgency on any single commit and nothing here blocks a dev. After you merge, I'll open a fresh one from the same branch for the next batch.
What lands here: new tickets and drafts, status/assignee changes, priority and dependency edits, ticket descriptions, verification notes when I close something.
If anything in here is contentious it'll say so in the ticket body rather than sitting in the PR description, so the reasoning stays with the ticket after merge.
Currently included
DRAFT-14 — per-group transformer fitting,
transformer(x) OVER (PARTITION BY g)AmirHossein's proposal, routed via Wren from TASK-3 brainstorming. Wren's measurements are recorded verbatim in the ticket since they're what make it scopeable.
The reframe that matters: a transformer ref is fitted once globally today — measured,
country a={10,20}, b={30,50}givesavg_age=[27.5], not per-country 15/40. That's the correct semantic (matches sklearn, where a Pipeline step fits once on all training data), so this is a new feature, not a bug.The split is the spine of the ticket:
PARTITION BYyields one state row per group,batch == infer, unseen groups return NULL for free.Four questions need your ruling before it becomes a task — unseen-group policy for (b), shorthand vs standard
PARTITION BYsyntax, whether (a) ships alone first, and artifact size at high group cardinality.Flagged as distinct from DRAFT-11: that one is argument binding, this is what data a transformer is fitted on.
🤖 Generated with Claude Code