Skip to content

docs: floor MGE Gaussians at a tenth of the pixel scale - #61

Merged
Jammy2211 merged 1 commit into
mainfrom
feature/mge-sigma-min-workspace-sweep
Aug 4, 2026
Merged

docs: floor MGE Gaussians at a tenth of the pixel scale#61
Jammy2211 merged 1 commit into
mainfrom
feature/mge-sigma-min-workspace-sweep

Conversation

@Jammy2211

Copy link
Copy Markdown
Collaborator

Phase 2 of the sweep started in autolens_workspace#467 (merged).

chapter_2_modeling/tutorial_5_linear_profiles.py builds three MGE sigma ladders by hand, all previously hardcoded to -2. They now start at np.log10(dataset.pixel_scales[0] / 10.0), so the basis stops spending Gaussians ~2 dex below what the data resolves.

All three are image-plane galaxy light. The companion HowToLens PR moves one of its ladders to -4 instead, because a lensed source is sampled far more finely than the image pixel scale — HowToGalaxy has no lensing and so no source plane, making that exclusion inapplicable here.

Scripts Changed

  • scripts/chapter_2_modeling/tutorial_5_linear_profiles.py (+ regenerated notebook)

Validation

  • pyflakes: zero introduced undefined names
  • Tutorial passes smoke (15s)
  • Notebook regenerated 1-for-1, no unrelated churn

Merge gate

pending-releasesigma_min exists only on PyAutoGalaxy main (#549, 13d3023c) and in no released PyAutoGalaxy.

🤖 Generated with Claude Code

Phase 2 of the sweep started in autolens_workspace#467. Tutorial 5's three
hand-rolled MGE ladders now start at
`np.log10(dataset.pixel_scales[0] / 10.0)` instead of a hardcoded -2.

All three are image-plane galaxy light — HowToGalaxy has no lensing and so
no source plane, making the source exclusion that applies in HowToLens
inapplicable here.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@Jammy2211 Jammy2211 added the pending-release PR queued for the next release build label Aug 4, 2026
@Jammy2211
Jammy2211 merged commit 51eed3d into main Aug 4, 2026
6 checks passed
@Jammy2211
Jammy2211 deleted the feature/mge-sigma-min-workspace-sweep branch August 4, 2026 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pending-release PR queued for the next release build

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant