Repository navigation
Native CPU transforms by shape and one forward pack (#584, #587, #595) - #598
Conversation
…a probe switch Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…s timed again Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…hape rule Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
John Travers accepted SEP 47 with five decisions taken before implementation: device capabilities keep their per-vendor status (CUDA low-storage is the only new qualification); every periodic statistic is computed natively; the native session owns the monitor's decision state and records, with exact identity on scalar decision logic; the reference session is built independently in phase 2; and #584 is delivered separately by PR #598. Part I also now specifies running checkpoint contents, monitor restoration, the calibration timing probe, groups as sequences of member problems and the A/B/C attribution. The roadmap entry moves to Accepted. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XGENPwVEY6NQKGaDDptaj9
Documentation build overview
17 files changed ·
|
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…s, 2 min and 7 min (#595) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… record Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Codex Review Summary: ✅ CompletedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7fed8b5ed1
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
…achable branches Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Native CPU FFT optimisation. ADR-075 is proposed for the maintainer.
Closes #584
Closes #587
Closes #595
What changes
fft_threshold, it takes a thread perfft_thread_elementssamples, at most one per row, or up to the medium width when its rows are at leastfft_long_rowlong.fft_fullis a backstop.benchmarks.md).SOLPROP_NATIVE_FORWARD_PACK=0turns the pack off (documented in the tuning guide)."fft"is back ingrain.TIMED, last in the order, with its threshold andfft_thread_elementshalved and doubled._core.time_fft_plan) and its ratio is stored beside its verdict.Results
Against
main: 5 % windows, three rounds, fields within 1e-12, withSOLPROP_CALIBRATION=offon both arms.argon_blueshiftctcrdwlineOn #595, measured plans for threaded rows (
SOLPROP_FFTW_PLAN=measure) win in the step:kerr_beam_1920.54–0.79).grin_nonlinear0.83).But planning them adds up to 12 s of construction (
ctcon the EPYC: 13.8 s against 1.6 s), so the default stays at estimate.The calibration's planning candidates "rarely won" because they were rarely timed. On all three machines the patient capture never finished, and the timing in the step ran out of rounds before a verdict.
maindoes the same on the EPYC: no verdict at thorough, and nothing planned at standard.With the pruning and the new budgets, the stage reaches a verdict on the EPYC 7763. Estimate's serial rows and planes were pruned (2.11 and 1.26 alone), and
threaded_rows measurescored 0.99 on the calibration's problems but tied a round, so the policy held.The thorough and standard calibrations on the EPYC 7763 conclude as on
main, admitting the copy threshold. The transforms' thread grain kept 4096 (2048: 0.976, not admitted; 8192: 1.13).Not done here
gnlse_raman_shockscales poorly with threads: slower at 16 than at 1 on an EPYC 9B45, and varies between processes #81's per-process modes, which still dominate the one-column cases on the Xeon.🤖 Generated with Claude Code