Skip to content

revert: drop PYAUTO_SKIP_LATENTS=1 (PyAutoLens #557 fixes upstream)#33

Merged
Jammy2211 merged 1 commit into
mainfrom
revert/skip-latents-now-fixed-upstream
May 28, 2026
Merged

revert: drop PYAUTO_SKIP_LATENTS=1 (PyAutoLens #557 fixes upstream)#33
Jammy2211 merged 1 commit into
mainfrom
revert/skip-latents-now-fixed-upstream

Conversation

@Jammy2211

Copy link
Copy Markdown
Contributor

Summary

PR #30 added `PYAUTO_SKIP_LATENTS=1` as a workaround for the magzero-required µJy latents that crashed the first A100 submit (job 322548). Upstream PyAutoLens commit `dd3d09b4` ("fix(latent): raw-flux latents + soft-fail magzero-required µJy", PR PyAutoLens#557) fixes the root cause:

  • Replaces `_require_magzero` with `_maybe_magzero_warn` — the three µJy latents now return `xp.nan` + one warning per process when `magzero` is missing, instead of raising.
  • Adds three raw-flux latents (`total_lens_flux`, `total_lensed_source_flux`, `total_source_flux`) that need no instrument inputs and default-on.

Search profiling now exercises the full first-class latent pipeline as intended.

Test plan

  • Resubmit A100 with `HPCPullPyAuto` having pulled the fix; confirm the search completes, post-fit latent step succeeds, and `latent.csv` lands.

🤖 Generated with Claude Code

PR #30 added PYAUTO_SKIP_LATENTS=1 as a workaround for the magzero-required
µJy latents that crashed the first A100 submit. Upstream PyAutoLens commit
dd3d09b4 ("fix(latent): raw-flux latents + soft-fail magzero-required µJy",
PR #557) replaces the _require_magzero raise with a soft NaN + one warning
per process, and adds three magzero-free raw-flux latents that default-on.

Search profiling now exercises the full first-class latent pipeline as
intended.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Jammy2211
Jammy2211 merged commit 4130dc2 into main May 28, 2026
1 check failed
@Jammy2211
Jammy2211 deleted the revert/skip-latents-now-fixed-upstream branch May 28, 2026 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant