search

tasoc / photometry

The TASOC Photometry Pipeline
https://tasoc.dk
GNU General Public License v3.0
18 stars 4 forks source link

Smarter loading of SPICE kernels #35

Closed rhandberg closed 2 years ago

rhandberg commented 2 years ago

Instead of loading all SPICE kernels into memory, we could limit the kernels that are loaded to only the ones needed for the particular sector being processed. That way we avoid the number of kernels needed to be downloaded and loaded into memory to keep increasing as the TESS mission continues.

codecov[bot] commented 2 years ago

Codecov Report

Merging #35 (2a35581) into devel (cc16209) will increase coverage by 0.05%. The diff coverage is 96.15%.

:exclamation: Current head 2a35581 differs from pull request most recent head da35573. Consider uploading reports for the commit da35573 to get more accurate results

@@            Coverage Diff             @@
##            devel      #35      +/-   ##
==========================================
+ Coverage   84.33%   84.39%   +0.05%     
==========================================
  Files          33       33              
  Lines        4366     4389      +23     
==========================================
+ Hits         3682     3704      +22     
- Misses        684      685       +1
Impacted Files Coverage Δ
photometry/spice.py 98.03% <95.91%> (-1.20%) :arrow_down:
photometry/BasePhotometry.py 93.92% <100.00%> (+<0.01%) :arrow_up:
photometry/version.py 50.70% <0.00%> (+1.40%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 760d3eb...da35573. Read the comment docs.