From 28c84710c1836446f69e9c4becabf9e7374f829f Mon Sep 17 00:00:00 2001 From: James Nightingale Date: Sun, 23 Aug 2026 22:24:16 +0000 Subject: [PATCH 1/2] docs: drop pynufft from the source-install optional requirements MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The install doc told users to `pip install pynufft` "for unit tests to pass". pynufft is no longer a dependency of any PyAuto library (PyAutoArray#475 deleted TransformerNUFFTPyNUFFT and dropped pynufft from both `optional` and `dev`), and PyAutoCTI's suite does not need it. Verified rather than assumed: - .github/workflows/main.yml calls PyAutoHeart's reusable lib-tests.yml, which installs ./PyAutoCTI[optional] plus the Nerves/Fit/Array chain and never installs pynufft in either leg. - pyproject.toml's `optional` extra is ["numba"] — pynufft is not in it. - The latest main run of that workflow is green on every leg: unittest on Python 3.12 and 3.13, plus the unittest-nojax leg. - A full-tree grep finds pynufft only in paper/paper.bib and files/citations.tex (published-record material, deliberately out of scope) and in this line — no import, no test, no config. Phase 3 of 3 of the pynufft-removal residue sweep (PyAutoHands#258). Co-Authored-By: Claude Opus 5 Claude-Session: https://claude.ai/code/session_01Q7kFdoYcD6wnTifNXV5T98 --- docs/installation/source.rst | 1 - 1 file changed, 1 deletion(-) diff --git a/docs/installation/source.rst b/docs/installation/source.rst index 8f038569..71c5fca5 100644 --- a/docs/installation/source.rst +++ b/docs/installation/source.rst @@ -55,7 +55,6 @@ For unit tests to pass you will also need the following optional requirements: .. code-block:: bash - pip install pynufft pip install pylops==1.11.1 If you are using a ``conda`` environment, add the source repository as follows: From 8c562a425ed18ca30bd59ae2b8b14e94fb5be8fa Mon Sep 17 00:00:00 2001 From: James Nightingale Date: Sun, 23 Aug 2026 22:35:53 +0000 Subject: [PATCH 2/2] docs: drop the defunct pylops from the source-install requirements MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Removes the whole "For unit tests to pass you will also need the following optional requirements" stanza from the Building-Only-PyAutoCTI section: with pynufft gone in the previous commit, `pip install pylops==1.11.1` was the only line left, and pylops is defunct. Same evidence as for pynufft: nothing in this repo imports or references pylops (the only other hit is paper/paper.bib, a published record), pyproject's `optional` extra is ["numba"], and CI installs no pylops while 270 of 271 tests pass — the single failure is an unrelated autoarray mask-shape guard on main. The sentence and its code-block go together; leaving the heading with an empty code-block would be invalid RST. Co-Authored-By: Claude Opus 5 Claude-Session: https://claude.ai/code/session_01Q7kFdoYcD6wnTifNXV5T98 --- docs/installation/source.rst | 6 ------ 1 file changed, 6 deletions(-) diff --git a/docs/installation/source.rst b/docs/installation/source.rst index 71c5fca5..c00238b2 100644 --- a/docs/installation/source.rst +++ b/docs/installation/source.rst @@ -51,12 +51,6 @@ Next, install the optional dependency numba via pip (see `this link