Skip to content

fix(smoke): extend latent_variables smoke to new 8-key registry #133

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Jammy2211 merged 1 commit into from
May 28, 2026
Merged

fix(smoke): extend latent_variables smoke to new 8-key registry #133

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 10 additions & 2 deletions config/latent.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,17 @@
# Workspace overrides for the library lensing latent toggles. Enables
# all five default latents so the smoke test exercises every entry in
# `autolens.analysis.latent.LATENT_FUNCTIONS`. See
# every entry in `autolens.analysis.latent.LATENT_FUNCTIONS` so the smoke
# test exercises the full registry. See
# `scripts/latent/latent_variables_smoke.py`.
total_lens_flux: true
total_lensed_source_flux: true
total_source_flux: true
total_lens_flux_mujy: true
total_lensed_source_flux_mujy: true
total_source_flux_mujy: true
magnification: true
effective_einstein_radius: true

# autogalaxy library default (loaded by autoconf into the same `latent`
# conf node) — explicitly disabled here to silence the cross-library
# "unknown latent" warning that would otherwise fire on every fit.
total_galaxy_0_flux: false
8 changes: 7 additions & 1 deletion scripts/latent/latent_variables_smoke.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,10 @@
and asserts every default-enabled key in ``autolens.analysis.latent.LATENT_FUNCTIONS``
is present with a finite value in a loose order-of-magnitude bracket.

The workspace ``config/latent.yaml`` enables all five library latents.
The workspace ``config/latent.yaml`` enables every entry in
``LATENT_FUNCTIONS`` — three raw-flux latents (instrument-input-free),
three µJy variants (require ``magzero``, which the smoke fixture
supplies), plus ``magnification`` and ``effective_einstein_radius``.
The brackets are deliberately loose so PYAUTO_SMALL_DATASETS grid mutations
(15x15) still yield passing values; the goal is a structural regression
guard, not Bayesian validation.
Expand Down Expand Up @@ -43,6 +46,9 @@
result = search.fit(model=model, analysis=analysis)

EXPECTED_KEYS = [
"total_lens_flux",
"total_lensed_source_flux",
"total_source_flux",
"total_lens_flux_mujy",
"total_lensed_source_flux_mujy",
"total_source_flux_mujy",
Expand Down
Loading