Please merge after PR #299, since it includes its changes.
Remaining issues:
[x] Inconsistent handling of missing data in the input image between the photutils and sep photometry modules.
[x] Inconsistent handling of missing image scale in the input image between the photutils and sep photometry modules.
[x] (Inconsistent) use of loop.run_in_executor() between the photutils and sep photometry modules. It is unclear why they are used at all.
Since most of the functionality could be extracted into separate classes, resolving these issues would also allow the duplication of code between the two modules to be further reduced through abstraction or dependency injection.
Please merge after PR #299, since it includes its changes.
Remaining issues:
loop.run_in_executor()between the photutils and sep photometry modules. It is unclear why they are used at all.Since most of the functionality could be extracted into separate classes, resolving these issues would also allow the duplication of code between the two modules to be further reduced through abstraction or dependency injection.