kevin218 / Eureka

Eureka! is a data reduction and analysis pipeline intended for time-series observations with JWST.
https://eurekadocs.readthedocs.io/
MIT License
58 stars 45 forks source link

HST ramp model and enabling calibrated spectra #534

Closed kevin218 closed 1 year ago

kevin218 commented 1 year ago

I wrote up the HST ramp model and am committing it here for others to test down the road.

I added some code that masks the uncalibrated part of the NIRSpec 2d array. It's quick and dirty, but it works. Once I get more experience with other instruments, I'll certainly want to make changes/enhancements. It's just not worth devoting more time until I know how the other instrument will work.

codecov-commenter commented 1 year ago

Codecov Report

Merging #534 (69109c7) into main (5c585fb) will decrease coverage by 0.25%. The diff coverage is 2.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##             main     #534      +/-   ##
==========================================
- Coverage   57.68%   57.43%   -0.25%     
==========================================
  Files          92       93       +1     
  Lines       11145    11195      +50     
==========================================
+ Hits         6429     6430       +1     
- Misses       4716     4765      +49     
Impacted Files Coverage Δ
...ureka/S5_lightcurve_fitting/models/HSTRampModel.py 0.00% <0.00%> (ø)
src/eureka/S3_data_reduction/s3_reduce.py 88.09% <16.66%> (-1.49%) :arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

kevin218 commented 1 year ago

Closing PR after merging #544