spacetelescope / stcal

https://stcal.readthedocs.io/en/latest/
Other
10 stars 32 forks source link

clean up unused arrays #277

Closed tapastro closed 1 month ago

tapastro commented 3 months ago

Partially addresses JP-3697

This PR addresses high memory usage in jump step by deleting some large arrays after use, and moving initialization of other arrays to the time at which they are needed.

Checklist

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.84%. Comparing base (2112773) to head (e7400b6). Report is 69 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #277 +/- ## ========================================== - Coverage 84.32% 82.84% -1.49% ========================================== Files 41 41 Lines 7529 7531 +2 ========================================== - Hits 6349 6239 -110 - Misses 1180 1292 +112 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

tapastro commented 1 month ago

Closing this - it appears that this breaks code in unexpected ways, and I don't have the time to tackle the full refactor.