Rectangular mesh split API update: RectangularAdapt* → RectangularBilinearAdapt* - #74
Merged
Conversation
…linearAdapt* PyAutoArray#462 / PyAutoGalaxy#579 / PyAutoLens#707 split the rectangular adaptive mesh family into Bilinear (rank-CDF, the workspace default) and RTU (kernel-CDF, advanced/interferometer) variants, retiring the old RectangularAdaptDensity / RectangularAdaptImage names. This is the HowToLens leg of that rollout, mirroring autolens_workspace's feature/rectangular-bilinear-rtu-mesh-split: every tutorial reference moves to the Bilinear default (per the no-workspace-example-uses-RTU decision recorded there), notebooks carry the same textual rename, and the prior configs are renamed with RTU twins added (same priors), matching that branch's config layout. Chapter-3 scripts + notebooks (12 tutorials), config/priors/mesh. The navigator catalogue does not embed the class names, so llms-full.txt and workspace_index.json are unchanged. Smoke-verified locally against current library mains (with the PyAutoArray fnnls JAX-coercion fix): the two tutorials red since 2026-08-20 (tutorial_8_adaptive_pixelization, tutorial_11_brightness_adaption) pass. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015GsUfbCwPd4XC8kpsiUJp7
tutorial_5_bayesian_formalism imports overlay_grid_from by its module path, which the mesh split also renamed: rectangular_adapt_density -> rectangular_rtu_adapt_density (where the function now lives). Caught by the full local run_smoke.py sweep (49/50 passed, this the one failure); the script now passes. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015GsUfbCwPd4XC8kpsiUJp7
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.
What
The HowToLens leg of the rectangular mesh split rollout (PyAutoArray#462 / PyAutoGalaxy#579 / PyAutoLens#707), which retired the
RectangularAdaptDensity/RectangularAdaptImagenames:RectangularBilinearAdaptDensity/RectangularBilinearAdaptImage— the workspace default, per the no-workspace-example-uses-RTU decision recorded on autolens_workspace'sfeature/rectangular-bilinear-rtu-mesh-split.config/priors/mesh/:rectangular_adapt_*.yamlrenamed torectangular_bilinear_adapt_*.yaml(keys updated) withrectangular_rtu_adapt_*.yamltwins added (same priors), matching that branch's config layout.llms-full.txt/workspace_index.jsonare unchanged.Why
mainsmoke has been red since 2026-08-20 22:30 UTC — first from the fnnls numba regression fixed in PyAutoArray#463, and since this morning's mesh-split merges every tutorial constructingRectangularAdaptDensityadditionally fails withAttributeError(confirmed on the main re-run). This red is theHowToLens: Smoke Tests failure on mainthat blocked the nightly release.This PR's smoke run picks up PyAutoArray's same-named
claude/wake-up-22piwpfix branch via the workflow's branch matching, so the two fixes are validated together; PyAutoArray#463 should merge first.Validation
tutorial_8_adaptive_pixelization+tutorial_11_brightness_adaption(the two originally-red scripts) pass locally against current library mains + PyAutoArray#463.run_smoke.pysweep across all 57 scripts running at time of writing; CI on this PR is the authoritative gate.🤖 Generated with Claude Code
https://claude.ai/code/session_015GsUfbCwPd4XC8kpsiUJp7
Generated by Claude Code