Repository navigation
Run the street gate with the Python CI actually installs - #7
Merged
Merged
Conversation
check:streets called .venv/bin/python. Actions installs Pillow and NumPy into setup-python and never creates that venv, so npm run verify exited 127 and every later gate was skipped. The runner now uses the illustration gate's resolver. The street scenes and their byte check are unchanged. Co-authored-by: Dr Non Arkara <nonsmartcity@gmail.com>
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.
Root cause
npm run verifyhas been red sincede075e6because that commit addedcheck:streetsas.venv/bin/python scripts/illustrations/build_streets.py --check. GitHub Actions installs Pillow and NumPy withpython -m pip installinto the setup-python interpreter and never creates a project.venv. The step exits 127 (sh: 1: .venv/bin/python: not found) before the gate measures a picture, so build, check-generated, check:release, check:slop, check:layout, check:hierarchy, check:sentences, check:data, check:illustrations, check:spatial-sync and check:spatial were all skipped. Run 42363651428; the previous commit0aad578was green.falsifyon that run succeeded.The street scenes, their credit, and the byte check in
build_streets.py --checkare unchanged.check:streetsandbuild:streetsnow go throughscripts/run-streets.mjs, which uses the same interpreter the illustration gate already uses:PYTHONif set, otherwise the project.venv, otherwisepython3on PATH.What was checked locally
.venvis absent.npm run check:streetsmeasures the five scenes (bytes, masters, sizes, credit ink).npm run verify— 288 tests, typecheck./amend/,/exam/,/news/,/read/,/sources/,/topics/): 378 fresh measurements, every finding already written down. A full local sweep lost its Chrome renderer mid-run (fatal signal 5); that is an environment crash, not a baseline change. The full sweep still runs in this workflow.