Skip to content

Commit b6f12b1

Browse files
Jammy2211Jammy2211
authored andcommitted
all unit tests pass
1 parent c3900fb commit b6f12b1

File tree

5 files changed

+35
-20
lines changed

5 files changed

+35
-20
lines changed

autoarray/dataset/imaging/dataset.py

Lines changed: 13 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -112,14 +112,23 @@ def __init__(
112112
"""
113113
)
114114

115-
self.psf = psf
116-
117115
if psf is not None:
116+
117+
if use_normalized_psf:
118+
119+
psf.kernel._array = np.divide(
120+
psf.kernel._array, np.sum(psf.kernel._array)
121+
)
122+
118123
if psf_setup_state:
119124

120125
state = ConvolverState(kernel=psf.kernel, mask=self.data.mask)
121126

122-
self.psf = Convolver(kernel=psf.kernel, state=state, normalize=use_normalized_psf)
127+
psf = Convolver(
128+
kernel=psf.kernel, state=state, normalize=use_normalized_psf
129+
)
130+
131+
self.psf = psf
123132

124133
self.grids = GridsDataset(
125134
mask=self.data.mask,
@@ -546,7 +555,7 @@ def output_to_fits(
546555
self.data.output_to_fits(file_path=data_path, overwrite=overwrite)
547556

548557
if self.psf is not None and psf_path is not None:
549-
self.psf.output_to_fits(file_path=psf_path, overwrite=overwrite)
558+
self.psf.kernel.output_to_fits(file_path=psf_path, overwrite=overwrite)
550559

551560
if self.noise_map is not None and noise_map_path is not None:
552561
self.noise_map.output_to_fits(file_path=noise_map_path, overwrite=overwrite)

autoarray/fixtures.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,10 @@ def make_image_7x7():
119119

120120

121121
def make_psf_3x3():
122-
psf = aa.Array2D.no_mask(values=[[0.0, 0.5, 0.0], [0.5, 1.0, 0.5], [0.0, 0.5, 0.0]], pixel_scales=(1.0, 1.0))
122+
psf = aa.Array2D.no_mask(
123+
values=[[0.0, 0.5, 0.0], [0.5, 1.0, 0.5], [0.0, 0.5, 0.0]],
124+
pixel_scales=(1.0, 1.0),
125+
)
123126

124127
return aa.Convolver(kernel=psf)
125128

autoarray/operators/convolver.py

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -854,15 +854,10 @@ def convolved_image_via_real_space_np_from(
854854
"This may change the correctness of the PSF convolution."
855855
)
856856

857-
print(self.kernel)
858-
print(image_native)
859-
860857
convolve_native = scipy_convolve(
861858
image_native, self.kernel.native.array, mode="same", method="auto"
862859
)
863860

864-
print(convolve_native)
865-
866861
convolved_array_1d = convolve_native[state.mask.slim_to_native_tuple]
867862

868863
return Array2D(values=convolved_array_1d, mask=image.mask)

test_autoarray/dataset/imaging/test_dataset.py

Lines changed: 15 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -125,11 +125,13 @@ def test__from_fits():
125125
)
126126

127127
assert (dataset.data.native == np.ones((3, 3))).all()
128-
assert dataset.psf.native == pytest.approx((1.0 / 9.0) * np.ones((3, 3)), 1.0e-4)
128+
assert dataset.psf.kernel.native == pytest.approx(
129+
(1.0 / 9.0) * np.ones((3, 3)), 1.0e-4
130+
)
129131
assert (dataset.noise_map.native == 3.0 * np.ones((3, 3))).all()
130132

131133
assert dataset.pixel_scales == (0.1, 0.1)
132-
assert dataset.psf.mask.pixel_scales == (0.1, 0.1)
134+
assert dataset.psf.kernel.mask.pixel_scales == (0.1, 0.1)
133135
assert dataset.noise_map.mask.pixel_scales == (0.1, 0.1)
134136

135137
dataset = aa.Imaging.from_fits(
@@ -143,11 +145,13 @@ def test__from_fits():
143145
)
144146

145147
assert (dataset.data.native == np.ones((3, 3))).all()
146-
assert dataset.psf.native == pytest.approx((1.0 / 9.0) * np.ones((3, 3)), 1.0e-4)
148+
assert dataset.psf.kernel.native == pytest.approx(
149+
(1.0 / 9.0) * np.ones((3, 3)), 1.0e-4
150+
)
147151
assert (dataset.noise_map.native == 3.0 * np.ones((3, 3))).all()
148152

149153
assert dataset.pixel_scales == (0.1, 0.1)
150-
assert dataset.psf.mask.pixel_scales == (0.1, 0.1)
154+
assert dataset.psf.kernel.mask.pixel_scales == (0.1, 0.1)
151155
assert dataset.noise_map.mask.pixel_scales == (0.1, 0.1)
152156

153157

@@ -167,7 +171,7 @@ def test__output_to_fits(imaging_7x7, test_data_path):
167171
)
168172

169173
assert (dataset.data.native == np.ones((7, 7))).all()
170-
assert dataset.psf.native[1, 1] == pytest.approx(0.33333, 1.0e-4)
174+
assert dataset.psf.kernel.native[1, 1] == pytest.approx(0.33333, 1.0e-4)
171175
assert (dataset.noise_map.native == 2.0 * np.ones((7, 7))).all()
172176
assert dataset.pixel_scales == (0.1, 0.1)
173177

@@ -187,8 +191,8 @@ def test__apply_mask(imaging_7x7, mask_2d_7x7, psf_3x3):
187191
== 2.0 * np.ones((7, 7)) * np.invert(mask_2d_7x7)
188192
).all()
189193

190-
assert masked_imaging_7x7.psf.slim == pytest.approx(
191-
(1.0 / 3.0) * psf_3x3.slim.array, 1.0e-4
194+
assert masked_imaging_7x7.psf.kernel.slim == pytest.approx(
195+
(1.0 / 3.0) * psf_3x3.kernel.slim.array, 1.0e-4
192196
)
193197

194198
assert type(masked_imaging_7x7.psf) == aa.Convolver
@@ -280,7 +284,7 @@ def test__different_imaging_without_mock_objects__customize_constructor_inputs()
280284

281285
masked_dataset = dataset.apply_mask(mask=mask)
282286

283-
assert masked_dataset.psf.native == pytest.approx(
287+
assert masked_dataset.psf.kernel.native == pytest.approx(
284288
(1.0 / 49.0) * np.ones((7, 7)), 1.0e-4
285289
)
286290
assert (masked_dataset.data == np.array([1.0])).all()
@@ -310,5 +314,7 @@ def test__psf_not_odd_x_odd_kernel__raises_error():
310314
psf = aa.Convolver(kernel=kernel)
311315

312316
dataset = aa.Imaging(
313-
data=image, noise_map=noise_map, psf=psf,
317+
data=image,
318+
noise_map=noise_map,
319+
psf=psf,
314320
)

test_autoarray/inversion/inversion/test_abstract.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -188,7 +188,9 @@ def test__curvature_matrix_via_sparse_operator__includes_source_interpolation__i
188188

189189
image = aa.Array2D.no_mask(values=np.random.random((7, 7)), pixel_scales=1.0)
190190
noise_map = aa.Array2D.no_mask(values=np.random.random((7, 7)), pixel_scales=1.0)
191-
kernel = aa.Array2D.no_mask([[0.0, 1.0, 0.0], [1.0, 1.0, 1.0], [0.0, 1.0, 0.0]], pixel_scales=1.0)
191+
kernel = aa.Array2D.no_mask(
192+
[[0.0, 1.0, 0.0], [1.0, 1.0, 1.0], [0.0, 1.0, 0.0]], pixel_scales=1.0
193+
)
192194

193195
psf = aa.Convolver(kernel=kernel, pixel_scales=1.0)
194196

0 commit comments

Comments
 (0)