docs: complete the arcticpy install recipe - #26
Merged
Conversation
The recipe said "install it after numpy" and gave only the bare `pip install arcticpy` line. That is not enough to install arcticpy on a clean modern venv — three gaps, each reproduced against a clean build of 2.6: - setuptools/wheel — --no-build-isolation does not read arcticpy's build-system.requires, so build deps must already be present. Without setuptools: `BackendUnavailable: Cannot import 'setuptools.build_meta'`. Python 3.12+ venvs no longer ship setuptools by default. - cython — named nowhere in this file, though the build needs it. - scipy/matplotlib — --no-deps suppresses arcticpy's runtime dependencies, but arcticpy/__init__.py imports read_noise, which imports both at import time, so a successful build still fails at `import arcticpy`. Also corrects the verification: arcticpy exposes no __version__ attribute, so `arcticpy.__version__` raises AttributeError on a healthy install. Points at PyAutoHeart/.github/actions/install-arcticpy as the recipe's single owner and the home of the arcticpy==2.6 pin. Refs PyAutoLabs/PyAutoHeart#170 Co-Authored-By: Claude Opus 5 <[email protected]> Claude-Session: https://claude.ai/code/session_018nDAxBEavkzb6Zkz1cYHef
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
2 participants
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.
Part of PyAutoLabs/PyAutoHeart#170. Docs-only leg.
AGENTS.md§arcticpy said "install it after numpy" and gave only the barepip install arcticpyline. That is not enough to install arcticpy on a clean modern venv.The gaps, each reproduced against a clean build of arcticpy 2.6
setuptools/wheel—--no-build-isolationdoes not read arcticpy'sbuild-system.requires, so build deps must already be present. Without setuptools:BackendUnavailable: Cannot import 'setuptools.build_meta'. Python 3.12+ venvs no longer ship setuptools by default.cython— named nowhere in this file, though the build needs it.scipy/matplotlib—--no-depssuppresses arcticpy's runtime dependencies, butarcticpy/__init__.pyimportsread_noise, which imports both at import time, so a successful build still fails atimport arcticpy.__version__attribute, soarcticpy.__version__raisesAttributeErroron a healthy install. Corrected toimportlib.metadata.The section now says why the two flags make the recipe fiddly, rather than only listing commands.
No CI change
This repo has no
.github/directory — it is a docs-only consumer of the recipe. It keeps pointing atPyAutoCTI/AGENTS.md§arcticpy for the full note (including the no-root header workaround, fixed in PyAutoLabs/PyAutoCTI#109), and now also namesPyAutoHeart/.github/actions/install-arcticpyas the recipe's single owner and the home of thearcticpy==2.6pin.Generated by Claude Code