cor estimator: add an option to skip flatfield
details
estimated_cor
fails to find the CoR on a .nx that contains no dark/flats frames. the do_flatfield
is indeed not handled.
close #568 (closed)
Edited by payno
Merge request reports
Activity
assigned to @payno
- Resolved by Pierre Paleo
added 1 commit
- 095a0081 - cor estimator: add an option to skip flatfield
added 1 commit
- f7e3837c - cor estimator: add an option to skip flatfield
requested review from @paleo
- Resolved by Pierre Paleo
Thanks, good catch.
CompositeCORFinder
is the only estimator not inheriting fromCORFinderBase
(technically it does, but it still does the flatfield and normalization by itself instead of using the parent method). One day it would be good to rewrite this estimator in a cleaner way.I also added
take_log
as a parameterEdited by Pierre Paleomentioned in commit 23b49f12
picked the changes into the branch
cherry-pick-23b49f12
with commit 782dd4d9mentioned in commit 782dd4d9
Please register or sign in to reply