Skip to content

Commit 1263577

Browse files
Jammy2211claude
authored andcommitted
Fix remaining Copilot review issues: unused import and duplicate assertion
- test_abstract.py: remove unused `import numpy as np`; fix duplicate assertion so second check covers row [1, :] not [0, :] again - test_fit_imaging.py: remove unused fixtures and dead setup block from covariance test (only masked_imaging_covariance_7x7 is needed) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent d5cbe62 commit 1263577

2 files changed

Lines changed: 3 additions & 12 deletions

File tree

test_autolens/imaging/test_fit_imaging.py

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -549,16 +549,8 @@ def test__figure_of_merit__sub_2_oversampling__linear_operated_profiles__inversi
549549

550550

551551
def test__figure_of_merit__sub_2_oversampling__covariance_noise_matrix__expected_log_likelihood(
552-
masked_imaging_covariance_7x7, image_7x7, psf_3x3, noise_map_7x7, mask_2d_7x7
552+
masked_imaging_covariance_7x7,
553553
):
554-
dataset = al.Imaging(
555-
data=image_7x7,
556-
psf=psf_3x3,
557-
noise_map=noise_map_7x7,
558-
over_sample_size_lp=2,
559-
)
560-
masked_imaging_7x7 = dataset.apply_mask(mask=mask_2d_7x7)
561-
562554
g0 = al.Galaxy(
563555
redshift=0.5,
564556
bulge=al.lp.Sersic(centre=(0.05, 0.05), intensity=1.0),

test_autolens/point/fit/positions/image/test_abstract.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
import numpy as np
21
import pytest
32

43
import autolens as al
@@ -41,6 +40,6 @@ def test__abstract_fit_positions_image_pair__multi_plane_tracer__model_positions
4140
assert fit_0.model_data[0, :] == pytest.approx(
4241
scaling_factor * fit_1.model_data.array[0, :], 1.0e-1
4342
)
44-
assert fit_0.model_data[0, :] == pytest.approx(
45-
scaling_factor * fit_1.model_data.array[0, :], 1.0e-1
43+
assert fit_0.model_data[1, :] == pytest.approx(
44+
scaling_factor * fit_1.model_data.array[1, :], 1.0e-1
4645
)

0 commit comments

Comments
 (0)