Skip to content

Return no operated_mapping_matrix_override when psf is None - #578

Merged
Jammy2211 merged 1 commit into
mainfrom
feature/interferometer-operated-override
Aug 21, 2026
Merged

Return no operated_mapping_matrix_override when psf is None#578
Jammy2211 merged 1 commit into
mainfrom
feature/interferometer-operated-override

Conversation

@Jammy2211

Copy link
Copy Markdown
Collaborator

Companion to PyAutoLabs/PyAutoArray#460 (PyAutoLabs/PyAutoArray#459).

PyAutoArray interferometer inversions now read operated_mapping_matrix_override. Interferometer fits construct LightProfileLinearObjFuncList with psf=None, where the image-space PSF-based override does not apply — and evaluating it would crash on psf.convolve_over_sample_size.

Changes

  • profiles/light/linear/abstract.pyoperated_mapping_matrix_override returns None when self.psf is None, keeping the standard mapping_matrix → transformer path for both ordinary and operated linear light profiles. Existing interferometer behaviour is preserved exactly.
  • Test covering both the ordinary (lp_linear.Sersic) and operated (lp_linear_operated.Gaussian) cases with psf=None.

Merge together with PyAutoLabs/PyAutoArray#460 — this guard must land no later than the autoarray change.

Test results

test_autogalaxy/: 1112 passed.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Lv2LDovvSQydk8biNMSAPL


Generated by Claude Code

PyAutoArray interferometer inversions now read
operated_mapping_matrix_override (PyAutoArray#459). Interferometer fits
construct LightProfileLinearObjFuncList with psf=None, where the
image-space PSF-based override below does not apply (and would crash on
psf.convolve_over_sample_size): returning None keeps the standard
mapping_matrix -> transformer path for both ordinary and operated linear
light profiles, preserving existing interferometer behaviour.

Co-Authored-By: Claude <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Lv2LDovvSQydk8biNMSAPL
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.

2 participants