Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
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
2 changes: 1 addition & 1 deletion .pre-commit-config.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
repos:
# Using this mirror lets us use mypyc-compiled black, which is about 2x faster
- repo: https://github.com/psf/black-pre-commit-mirror
rev: 24.1.1
rev: 26.1.0
hooks:
- id: black
# It is recommended to specify the latest version of Python
Expand Down
6 changes: 2 additions & 4 deletions tests/test_calibration/test_cal_auto_ratio_remultiply.py
Original file line number Diff line number Diff line change
Expand Up @@ -96,10 +96,8 @@ def test_cal_auto_ratio_remultiply(before_file, after_file):
and then calls `cal_auto_ratio_remultiply`, checking the outputs match expectations
"""
if before_file == None or after_file == None:
pytest.skip(
f"""This test has been skipped because the test was listed in the
skipped tests due to FHD not outputting them: {skip_tests}"""
)
pytest.skip(f"""This test has been skipped because the test was listed in the
skipped tests due to FHD not outputting them: {skip_tests}""")

h5_before = load(before_file)
expected_cal = load(after_file)
Expand Down
6 changes: 2 additions & 4 deletions tests/test_calibration/test_calibrate_qu_mixing.py
Original file line number Diff line number Diff line change
Expand Up @@ -80,14 +80,12 @@ def test_qu_mixing(before_file, after_file):
"""Runs the test on `calibrate_qu_mixing` - reads in the data in `data_loc`,
and then calls `calibrate_qu_mixing`, checking the outputs match expectations"""
if before_file == None or after_file == None:
pytest.skip(
f""""This test has been skipped because the test was listed
pytest.skip(f""""This test has been skipped because the test was listed
in the skipped tests due to FHD not outputting them: {skip_tests}.
In this case it as due to the LA_LEAST_SQUARES differences
compared to np.linalg.lstsq which doesn't use double precision
by default which for some reason makes a difference for values
close to 0 in single precision."""
)
close to 0 in single precision.""")

h5_before = load(before_file)
expected_calc_phase = load(after_file)
Expand Down
6 changes: 2 additions & 4 deletions tests/test_calibration/test_vis_calibration_flag.py
Original file line number Diff line number Diff line change
Expand Up @@ -101,16 +101,14 @@ def test_vis_calibration_flag(before_file, after_file):
"""Runs the test on `vis_calibration_flag` - reads in the data in before_file and after_file,
and then calls `vis_calibration_flag`, checking the outputs match expectations"""
if before_file == None or after_file == None:
pytest.skip(
f"""This test has been skipped because the test was listed in the
pytest.skip(f"""This test has been skipped because the test was listed in the
skipped tests due to FHD not outputting them: {skip_tests}. The reason
for skipping point_offzenith_run3 is due to the difference of NumPy median
and median in IDL. IDL doesn't take the average of the two middle numbers
when dealing with an an array of even values. It wasn't a problem in the other
cases, but not this one, and because we're dealing with a continuous set of data,
it doesn't feel appropriate to use the IDL median.
"""
)
""")

h5_before = load(before_file)
expected_obs = load(after_file)
Expand Down