Skip to content

docs: retire pynufft from wiki, skills and installation tables - #19

Merged
Jammy2211 merged 1 commit into
mainfrom
feature/pynufft-removal-residue-phase-2
Aug 23, 2026
Merged

docs: retire pynufft from wiki, skills and installation tables#19
Jammy2211 merged 1 commit into
mainfrom
feature/pynufft-removal-residue-phase-2

Conversation

@Jammy2211

Copy link
Copy Markdown
Contributor

Retires pynufft from this assistant's wiki, skills and installation tables. TransformerNUFFTPyNUFFT was deleted by PyAutoLabs/PyAutoArray#475; these pages still documented it as an available non-JAX fallback, so an assistant answering from them would hand a user a class that raises AttributeError.

Files Changed

File Change
skills/ag_build_interferometer_model.md Drop the ag.TransformerNUFFTPyNUFFT bullet
skills/ag_setup_environment.md pynufftnufftax
wiki/core/concepts/interferometer_theory.md Drop the legacy-transformer bullet
wiki/core/api/datasets.md Replace the legacy sentence; name TransformerDFT as the no-JAX path
wiki/core/operations/installation.md Remove pynufft from the optional table row; rewrite the standalone note
wiki/core/stack/autoarray.md, wiki/core/stack/autogalaxy.md Drop pynufft from the dependency lists

Extras corrected against the real source, not just de-pynufft'd

PyAutoGalaxy's actual optional extra is autogalaxy[jax], numba, zeus-mcmc, getdist. pynufft is gone — but nufftax was never in it either (it lives in PyAutoArray's optional). The old note implied the extra provided the NUFFT backend. installation.md now says to install nufftax explicitly.

Provenance

The 5 edited wiki/core pages were re-stamped with --write-provenance --page (scoped, so no unrelated page's pin moved). --check-provenance is clean.

Pre-existing failure, untouched

--check-citations reports wiki/core/operations/sandbox.md citing PyAutoGalaxy:autogalaxy/plot/plot_utils.py, which no longer exists. That file is not touched by this PR (git diff confirms) and the path is genuinely gone from PyAutoGalaxy main. Left for separate triage rather than folded in here.

Note for the reviewer

Heart was RED (release validation FAILED, unrelated) at PR-open; the human explicitly authorized push + PR-open. Merge remains gated.

Refs PyAutoLabs/autogalaxy_workspace#224

PyAutoArray#475 deleted TransformerNUFFTPyNUFFT; these pages still documented
it as an available non-JAX fallback and listed pynufft in the optional extras.

Extras tables corrected against the real PyAutoGalaxy pyproject.toml, whose
'optional' is autogalaxy[jax], numba, zeus-mcmc, getdist -- pynufft is gone and
nufftax was never in it, so installation.md now says to install nufftax
explicitly rather than implying the extra provides it.

Where a non-JAX fallback is genuinely needed (Intel macOS, which has no JAX
wheels) the docs now point at TransformerDFT.

The 5 edited wiki/core pages were re-stamped with --write-provenance --page
(scoped, so no unrelated page's pin moved); --check-provenance is clean.

NOTE: --check-citations reports a pre-existing failure in
wiki/core/operations/sandbox.md (PyAutoGalaxy:autogalaxy/plot/plot_utils.py no
longer exists). Untouched by this change and left for separate triage.

Refs PyAutoLabs/autogalaxy_workspace#224

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JEXzQpvG3QNUdTh6tZcaAE
@Jammy2211
Jammy2211 merged commit 6dbb15e into main Aug 23, 2026
1 of 2 checks passed
@Jammy2211
Jammy2211 deleted the feature/pynufft-removal-residue-phase-2 branch August 23, 2026 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant