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.

Sign up for GitHub

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

Jump to bottom

Move common resample code to stcal #8695

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

remove bad assert

0c18d77
Select commit
Loading
Failed to load commit list.
Draft

Move common resample code to stcal #8695

remove bad assert
0c18d77
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project failed Oct 11, 2024 in 0s

55.45% (-6.34%) compared to a84347b

View this Pull Request on Codecov

55.45% (-6.34%) compared to a84347b

Details

Codecov Report

Attention: Patch coverage is 4.13793% with 278 lines in your changes missing coverage. Please review.

Project coverage is 55.45%. Comparing base (a84347b) to head (0c18d77).
Report is 7 commits behind head on main.

Files with missing lines Patch % Lines
jwst/resample/resample.py 3.33% 145 Missing ⚠️
jwst/resample/resample_step.py 3.70% 104 Missing ⚠️
jwst/resample/resample_spec.py 0.00% 16 Missing ⚠️
jwst/resample/resample_utils.py 22.22% 7 Missing ⚠️
jwst/resample/resample_spec_step.py 16.66% 5 Missing ⚠️
jwst/resample/gwcs_drizzle.py 0.00% 1 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #8695      +/-   ##
==========================================
- Coverage   61.79%   55.45%   -6.34%     
==========================================
  Files         377      377              
  Lines       38832    39012     +180     
==========================================
- Hits        23997    21635    -2362     
- Misses      14835    17377    +2542

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

View more details on Codecov