Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removed redundant calculations, unneeded array allocations from jump step #302

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from

Merge branch 'spacetelescope:main' into jumpdetection_speedup

412f078
Select commit
Loading
Failed to load commit list.
Draft

Removed redundant calculations, unneeded array allocations from jump step #302

Merge branch 'spacetelescope:main' into jumpdetection_speedup
412f078
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Dec 23, 2024 in 1s

85.71% of diff hit (target 29.57%)

View this Pull Request on Codecov

85.71% of diff hit (target 29.57%)

Annotations

Check warning on line 201 in src/stcal/jump/twopoint_difference.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/stcal/jump/twopoint_difference.py#L201

Added line #L201 was not covered by tests

Check warning on line 204 in src/stcal/jump/twopoint_difference.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/stcal/jump/twopoint_difference.py#L204

Added line #L204 was not covered by tests

Check warning on line 207 in src/stcal/jump/twopoint_difference.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/stcal/jump/twopoint_difference.py#L207

Added line #L207 was not covered by tests

Check warning on line 211 in src/stcal/jump/twopoint_difference.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/stcal/jump/twopoint_difference.py#L211

Added line #L211 was not covered by tests

Check warning on line 217 in src/stcal/jump/twopoint_difference.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/stcal/jump/twopoint_difference.py#L213-L217

Added lines #L213 - L217 were not covered by tests

Check warning on line 443 in src/stcal/jump/twopoint_difference.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/stcal/jump/twopoint_difference.py#L443

Added line #L443 was not covered by tests