SciML / SciML/SparseWithDenseRowColMatrices.jl

Tracking: remove unregistered-dep workarounds once PureKLU + SparseColumnPivotedQR are registered; revisit Aqua tmax

Open
#17 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Julia
Stars
0
Forks
1
Avg merge
14m
Merged PRs (30d)
6

Description

WHAT: CI.yml:32-36 develops both deps from git URLs on every matrix entry because neither is in any reachable registry (confirmed) — this and the install-doc workaround are the only things letting the package resolve. The [compat] bounds PureKLU="0.1" / SparseColumnPivotedQR="0.1" (Project.toml:29,33) are not registry-enforced until registration, so a breaking change in csr_qr/csr_refactor!/klu! signatures won't be caught by compat. Separately, test/runtests.jl:55-58 raises the Aqua persistent_tasks budget 6x (10s→60s) to work around precompile-contention flakiness — a band-aid that could also hide a genuine future load slowdown. Note: there is NO [sources] block in Project.toml today (only the CI step and README need cleanup). WHY IT MATTERS: Blocked on external registration; folds several low-priority cleanups into one tracking item. FIX: When registration lands — delete the 'Develop unregistered SciML dependencies' CI step, simplify README install to a normal Pkg.add, confirm/tighten the 0.1 compat bounds (let CompatHelper maintain them), verify a clean resolve from General, and re-test whether the default Aqua tmax passes (lower/remove the 60s budget). EFFORT: S.


Priority: low. Filed from an automated next-steps audit of the QR/lstsq work (see PR #6).

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

After PureKLU and SparseColumnPivotedQR are registered, inspect CI.yml:32-36, Project.toml:29,33, README, and test/runtests.jl:55-58. Confirm a clean General resolve, remove the unregistered-dependency and install workarounds, review the compat bounds, and verify whether Aqua passes with its default persistent_tasks budget.

Written by the indexing model from the issue text.

Assessment

Tech stack
julia
Domain
build-system, ci-cd, documentation, testing
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.