Skip to content

Commit 24db1ba

Browse files
igerberclaude
andcommitted
Add deferred Rust backend optimizations to TODO.md
Documents post-merge optimization opportunities from PR #58 review: - Matrix inversion efficiency (Cholesky) - Reduce bootstrap allocations - Consider static BLAS linking Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent 2e580c5 commit 24db1ba

1 file changed

Lines changed: 11 additions & 1 deletion

File tree

TODO.md

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,9 +102,19 @@ From code review (PR #32):
102102

103103
---
104104

105+
## Rust Backend Optimizations
106+
107+
Deferred from PR #58 code review (can be done post-merge):
108+
109+
- [ ] **Matrix inversion efficiency** (`rust/src/linalg.rs:180-194`): Use Cholesky factorization for symmetric positive-definite matrices instead of column-by-column solve
110+
- [ ] **Reduce bootstrap allocations** (`rust/src/bootstrap.rs`): Currently uses `Vec<Vec<f64>>` → flatten → `Array2` which allocates twice. Should allocate directly into ndarray.
111+
- [ ] **Consider static BLAS linking** (`rust/Cargo.toml`): Currently requires system BLAS libraries. Consider `openblas-static` or `intel-mkl-static` features for easier distribution.
112+
113+
---
114+
105115
## Performance Optimizations
106116

107-
No major performance issues identified. Potential future optimizations:
117+
Potential future optimizations:
108118

109119
- [ ] JIT compilation for bootstrap loops (numba)
110120
- [ ] Parallel bootstrap iterations

0 commit comments

Comments
 (0)