build: floor autonerves at 2026.8.23.1 for the SMALLDAT regime stamp - #481
Open
Jammy2211 wants to merge 1 commit into
Open
build: floor autonerves at 2026.8.23.1 for the SMALLDAT regime stamp#481Jammy2211 wants to merge 1 commit into
Jammy2211 wants to merge 1 commit into
Conversation
The previous floor, autonerves>=2026.8.22.1, was the newest release on PyPI but predates the SMALLDAT regime stamp (PyAutoNerves#153/#154). A PyPI-installed autoarray therefore resolved an autonerves whose FITS writer emits no card at all: should_simulate read the regime as "unknown" and fell back to the shape heuristic, which provably cannot see capped interferometer datasets -- the case the stamp exists for. The stamp was inert for anyone installing rather than running from a checkout. 2026.8.23.1 is the first release carrying it (nerves 0ecefa0). Verified against the published wheel in a clean venv, not the source tree: the card is written as a genuine FITS boolean in both regimes. The existing PyAutoLens#687/#702 reason for this floor is kept -- both reasons are load-bearing and the comment now says so. Deliberately unchanged: - the shape fallback in should_simulate and _is_capped_at_the_current_cap on the capped branch. This floor governs what a fresh install WRITES; it says nothing about datasets already on disk, every one of which is unstamped. Those fallbacks are what protect them. - the duplicated "SMALLDAT" literal in dataset_util. Importing it from autonerves becomes safe now the floor names a stamped release, but the duplication is documented and a stale reader degrades to the fallback, which is the safe direction. Co-Authored-By: Claude Opus 5 <[email protected]> Claude-Session: https://claude.ai/code/session_01QqMr6QKGALYhcxpQAHZtry
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.
Bumps
autoarray's floor fromautonerves>=2026.8.22.1to>=2026.8.23.1.Why
2026.8.22.1was the newest release on PyPI but predates the SMALLDAT regime stamp (PyAutoNerves#153/#154). A PyPI-installedautoarrayresolved anautonerveswhose FITS writer emits no card at all, soshould_simulateread the regime as "unknown" and fell back to the shape heuristic — which cannot see capped interferometer datasets, the case the stamp exists for.Nothing was broken by that; it is the designed degradation. But the stamp was inert for anyone installing rather than running from a checkout.
2026.8.23.1(nerves0ecefa0) is the first release carrying it.Verification
Checked against the published wheel in a clean venv with
PYTHONPATHstripped, not the source tree:A genuine FITS boolean, not a string —
bool("F")isTruein Python, so a coerced string card would invert the regime.Deliberately unchanged
should_simulate, and_is_capped_at_the_current_capon the capped branch. The floor governs what a fresh install writes; it says nothing about datasets already on disk, every one of which is unstamped. Those fallbacks protect them.PyAutoLens#687/#702comment. That explains a different historical reason for this floor (JAX moving into autonerves' base deps). Both reasons are load-bearing, so the stamp reason was added, not substituted — and the comment now says so explicitly."SMALLDAT"literal indataset_util. Importing it from autonerves becomes safe now the floor names a stamped release, but the duplication is documented and a stale reader degrades to the fallback, which is the safe direction. Converting it is optional and low value.Closes the follow-up filed as
PyAutoMind/draft/maintenance/libraries/bump_autoarray_autonerves_floor_after_stamp_release.md.