Skip to content

T24 — Partitioned-inverse fused solve (solve_alg = "algo1", issue #82 solve half) #24

Closed as not planned

Description

@michel2323

Task T24 of TASKS.md — milestone M11 — Performance. (Renamed on 2026-10-09: the ND partition-tree task moved to T30, see STATE.md §8.)

Depends on: #23 (T23) must be merged first; tasks are strictly sequential.

Summary (from TASKS.md): adopt the PR #107 solve prototype (bench/solve_proto_*.jl): per-front L₁₁ inverses turning every TRSV into a GEMV, one fused dependency-counter kernel per direction for fronts of width ≤ 256 (atomic-free variant kept), strategy chosen per schedule (it regresses on lap3d_40 and apache2), Cholesky and LDLᵀ, nrhs > 1; every solve suite repeated under "algo1" with a residual gate; Report: solve table vs the cuDSS baseline. Refs #82.

Scope and deliverables: the section T24 of TASKS.md on main is the single source of truth; it is not copied here so it cannot drift.

Definition of done:

  • Every test listed in the task passes on the KA CPU backend and on the CUDA self-hosted runner (Run tests and Aqua green on the PR).
  • The task's status marker in TASKS.md is set and its Report block is filled in honestly (CUDA counts come from CI).
  • A PR titled T24: … from branch task/T24-<slug> with Closes #24 in its body, reviewed by the Claude reviewer with VERDICT: APPROVE.
  • Out-of-scope bugs or follow-ups found on the way are opened as separate issues (label found-by-agent), not fixed silently.

How it starts: add the label claude:implement (the pipeline does this automatically after the previous task's PR merges; label on-hold to prevent that).

Activity

  1. changed the title [-]T24 — ND partition-tree export/import and ordering cache[/-] [+]T24 — Partitioned-inverse fused solve (`solve_alg = "algo1"`, issue #82 solve half)[/+] on Oct 9, 2026
  2. github-actions commented on Oct 9, 2026

    @github-actions
    Contributor

    Previous task merged in #113; the Claude implementer was started for this issue by the pipeline.

  3. added
    on-holdDo not chain into this task automatically
    and removed
    claude:implementStart the Claude implementer on this task issue
    on Oct 9, 2026
  4. github-actions commented on Oct 9, 2026

    @github-actions
    Contributor

    The implementer's branch was published with the workflow token, so CI and the reviewer did not start on its PR: close and reopen the PR (or push to it) to start them. Set the repository secret CLAUDE_GH_PAT to avoid this.

  5. michel2323 commented on Nov 5, 2026

    @michel2323
    MemberAuthor

    Superseded: the task is marked [-] in TASKS.md since PR #185 (2026-11-04). Its kernels landed through T33–T35 in src/fused.jl; the remaining integration is re-cut as T36 (#186, solve_alg = "algo1" on DirectSolver), T37 (the fused factorization on DirectSolver, #109) and T38 (graph capture of the stock path, closes #82 and #110).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    on-holdDo not chain into this task automaticallytaskOne task from TASKS.md

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions