astropy / astropy/ccdproc

Performance: Combining flats is slow

Open
#645 2 comments 0 reactions 0 assignees View on GitHub
combiner effort-high package-expert performance
Dominant language
Python
Stars
93
Forks
92
Avg merge
14h 44m
Merged PRs (30d)
30

Description

This is taken from Astropy project performance survey. Is this is not the correct repo for this issue, please advise.


I have not tried this in recent astropy versions, but the early code to combine flats was FAR slower than IRAF `imcombine`. I had done some reading up to discover that IRAF was very proud about how it did memory management to do this efficiently. Sorry cannot be more specific or useful than that.

Contributor guide

Open the contributing guide

Research direction

The report names no implementation file, test, benchmark, or recent Astropy version. Start by locating the current flat-combination path in ccdproc and reproducing its performance with a representative dataset, then compare the result with IRAF imcombine. Done would require a defined benchmark and an agreed performance target, neither of which the issue currently specifies.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
performance
Issue type
Bug
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
18/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.