Precompile workload covers only the LU path — none of the QR / lstsq / adjoint surface is precompiled
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 75/100
- Issue type
- Feature
- Clarity
- Clearly specified
- Activity status
- Quiet
- Tech stack
- julia
- Domain
- performance
Research direction
Start in src/SparseWithDenseRowColMatrices.jl:37-53 and inspect the existing PrecompileTools workload inside the Float64 and ComplexF64 loop. Extend the workload to cover the named QR, adjoint/transpose matvec, and lstsq paths using the small dimensions described, then verify that the package's precompile workload completes successfully.
Written by the indexing model from the issue text.
Description
WHAT: The @compile_workload (src/SparseWithDenseRowColMatrices.jl:37-53) exercises only A*b, factorize(A), F\b, refactor!(F, nonzeros(...)) — all LU/Woodbury. It never touches qr, QR refactor!/ldiv!, adjoint/transpose matvec (A'*u), or lstsq/SparseWithDenseRowColLeastSquares. WHY IT MATTERS: The entire new feature surface — the point of this work — pays full first-call compilation latency, directly undercutting the new features' UX. PrecompileTools is already set up, so extending is cheap. FIX: In the existing for T in (Float64, ComplexF64) loop add Fq = qr(A); Fq \ b; refactor!(Fq, A); Fq' \ b; A' * b; lstsq(A2, b) (small well-posed A2, guard the lstsq call). Keep it small (current workload uses n=12, r=2). EFFORT: S.
Priority: medium. Filed from an automated next-steps audit of the QR/lstsq work (see PR #6).
- Dominant language
- Julia
- Stars
- 0
- Forks
- 1
- Avg merge
- 14m
- Merged PRs (30d)
- 6
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from SciML/SparseWithDenseRowColMatrices.jl
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
SciML/SparseWithDenseRowColMatrices.jl#25 · 1 comment ·
-
Difficulty 4/5 3-5 days Newbie friendliness 52/100
-
Difficulty 4/5 3-5 days Newbie friendliness 38/100
-
Difficulty 4/5 3-5 days Newbie friendliness 48/100
-
Difficulty 4/5 3-5 days Newbie friendliness 55/100
All issues in SciML/SparseWithDenseRowColMatrices.jl
Similar issues
-
bug 🐞 distributed 🕸️
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
CliMA/Oceananigans.jl#6049 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
-
Difficulty 1/5 Under an hour Newbie friendliness 88/100
-
Difficulty 1/5 1-3 hours Newbie friendliness 88/100
JuliaGPU/GPUArrays.jl#780 ·
-
Difficulty 2/5 Half a day Newbie friendliness 76/100
epiforecasts/EpiBranch.jl#304 ·