Skip to content

Commit

Permalink
fix codestyle bug; remove mark from fixture in test_line_matching
Browse files Browse the repository at this point in the history
  • Loading branch information
tepickering committed May 17, 2024
1 parent f226188 commit 1ed3c2c
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
2 changes: 1 addition & 1 deletion specreduce/line_matching.py
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ def find_arc_lines(
fwhm *= spectrum.spectral_axis.unit

if fwhm.unit != spectrum.spectral_axis.unit:
raise ValueError("fwhm must have the same units as the spectral axis of the input spectrum.")
raise ValueError("fwhm must have the same units as spectrum.spectral_axis.")

Check warning on line 57 in specreduce/line_matching.py

View check run for this annotation

Codecov / codecov/patch

specreduce/line_matching.py#L57

Added line #L57 was not covered by tests

detected_lines = find_lines_threshold(spectrum, noise_factor=noise_factor)
detected_lines = detected_lines[detected_lines['line_type'] == 'emission']
Expand Down
1 change: 0 additions & 1 deletion specreduce/tests/test_line_matching.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@
from specreduce.line_matching import match_lines_wcs, find_arc_lines


@pytest.mark.remote_data
@pytest.fixture
def mk_test_data():
"""
Expand Down

0 comments on commit 1ed3c2c

Please sign in to comment.