Skip to content

Use chunking in co-addition wherever possible to minimize excessive memory usage#610

Merged
astrofrog merged 10 commits into
astropy:mainfrom
astrofrog:coadd-chunked-combine
Jun 30, 2026
Merged

Use chunking in co-addition wherever possible to minimize excessive memory usage#610
astrofrog merged 10 commits into
astropy:mainfrom
astrofrog:coadd-chunked-combine

Conversation

@astrofrog

Copy link
Copy Markdown
Member

WIP

@codecov

codecov Bot commented Jun 28, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 83.82353% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.31%. Comparing base (9a48bac) to head (fedf1e3).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
reproject/mosaicking/_coadd.py 81.13% 10 Missing ⚠️
reproject/mosaicking/_subset_array.py 93.33% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #610      +/-   ##
==========================================
+ Coverage   88.13%   88.31%   +0.18%     
==========================================
  Files          51       51              
  Lines        2074     2089      +15     
==========================================
+ Hits         1828     1845      +17     
+ Misses        246      244       -2     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@astrofrog astrofrog changed the title Do co-addition in chunks to avoid excessive memory usage Use chunking in co-addition wherever possible to minimize excessive memory usage Jun 29, 2026
…nks method to make it easier to iterate over array subset chunks in co-addition
…handling, also avoid excessive memory usage for NaN -> 0 masking even when in background matching mode
…min/max, and factor 'summation' code out into a helper private function to avoid duplication
@astrofrog astrofrog force-pushed the coadd-chunked-combine branch from 98e8c77 to 544e7a3 Compare June 29, 2026 11:25
@astrofrog astrofrog marked this pull request as ready for review June 29, 2026 13:02
@astrofrog

Copy link
Copy Markdown
Member Author

This seems to work well, so I'll go ahead and merge

@astrofrog astrofrog merged commit fb280d2 into astropy:main Jun 30, 2026
26 of 27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant