Repository navigation
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>
jtravs
added a commit
that referenced
this pull request
Nov 9, 2026
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
11 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>
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Native CPU FFT optimisation, for #584, #587 and #595. Draft until reviewed; ADR-075 is proposed for the maintainer.
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. The thorough level no longer plans at patient. The calibration version is now 2.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. Giving the FFT stage enough time is a decision on the levels' budgets; see ADR-075.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