Skip to content

Commit

Permalink
Update changes
Browse files Browse the repository at this point in the history
  • Loading branch information
WilliamJamieson committed Nov 6, 2023
1 parent 0e54e3e commit ba5589c
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGES.rst
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ ramp_fitting
- Refactor Casertano, et.al, 2022 uneven ramp fitting and incorporate the matching
jump detection algorithm into it. [#215]
- Fix memory issue with uneven ramp fitting [#226]
- Fix some bugs in the jump detection algorithm [#227]

Changes to API
--------------
Expand Down
4 changes: 3 additions & 1 deletion tests/test_jump_cas22.py
Original file line number Diff line number Diff line change
Expand Up @@ -575,7 +575,9 @@ def test_find_jumps(jump_data):
# The two resultants excluded should be adjacent
jump_correct = []
for jump in fit['jumps']:
jump_correct.append(jump == resultant_index or jump == resultant_index - 1 or jump == resultant_index + 1)
jump_correct.append(jump == resultant_index or
jump == resultant_index - 1 or
jump == resultant_index + 1)
if not all(jump_correct):
incorrect_other += 1
continue

Check warning on line 583 in tests/test_jump_cas22.py

View check run for this annotation

Codecov / codecov/patch

tests/test_jump_cas22.py#L582-L583

Added lines #L582 - L583 were not covered by tests
Expand Down

0 comments on commit ba5589c

Please sign in to comment.