Skip to content

T15 — GPU LDLᵀ/LDLᴴ in regimes A/B/C, diagonal solve, statistics #15

Closed

Description

@michel2323

Task T15 of TASKS.md — milestone M4 — Symmetric indefinite LDLᵀ/LDLᴴ (MadNLP-ready).

Reads: PLAN §2.4, §3.3, §2.6 (sytrf), §1.4.

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

Scope and deliverables: the section T15 of TASKS.md on main is the single source of truth; it is not copied here so it cannot drift. Read it together with the PLAN.md sections it names and the Report blocks of the previous tasks.

Definition of done (tests: test/test_numeric_ldlt.jl, ported tests, CPU + CUDA):

  • Every test listed in the task passes on the KA CPU backend (GitHub runner) and on the CUDA/AMDGPU self-hosted runners (Run tests and Aqua checks 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 T15: … from branch task/T15-<slug> with Closes #15 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).


Generated by Claude Code

Activity

  1. github-actions commented on Oct 2, 2026

    @github-actions
    Contributor

    Previous task merged in #65, but T15 is held: open found-by-agent issues without the triaged label: #64. Label them triaged (or close them) and the pipeline starts this task automatically (claude-triage.yml).

  2. github-actions commented on Oct 2, 2026

    @github-actions
    Contributor

    Claude implement finished without opening a pull request (run: https://github.com/exanauts/SparseDirectSolver.jl/actions/runs/37032539796; pushed branch: none). The execution log is attached to the run as an artifact. Labelled needs-owner.

  3. removed
    needs-ownerThe Claude pipeline stopped; the owner must look
    on Oct 2, 2026
  4. michel2323 commented on Oct 2, 2026

    @michel2323
    MemberAuthor

    Recovered: the implementer's commit was printed into the execution log as a patch (the App token had expired after the 81-minute run). Reassembled, checksum verified, applied on main and pushed as task/T15-gpu-ldlt; PR #76. needs-owner removed.


    Generated by Claude Code

  5. github-actions commented on Oct 2, 2026

    @github-actions
    Contributor

    Merged in #76.

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

    claude:implementStart the Claude implementer on this task issuetaskOne task from TASKS.md

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions