Skip to content
Snippets Groups Projects

cor estimator: add an option to skip flatfield

Merged payno requested to merge fix_568 into 2024.2

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

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • payno added 1 commit

    added 1 commit

    • 095a0081 - cor estimator: add an option to skip flatfield

    Compare with previous version

  • payno added 1 commit

    added 1 commit

    • f7e3837c - cor estimator: add an option to skip flatfield

    Compare with previous version

  • payno marked this merge request as ready

    marked this merge request as ready

  • payno requested review from @paleo

    requested review from @paleo

  • Author Owner

    CI fails today because of cuda - unrelated.

    Tested on the blamed dataset. Works there.

  • payno
  • closed

  • reopened

  • payno changed the description

    changed the description

  • Pierre Paleo added 1 commit

    added 1 commit

    Compare with previous version

  • Thanks, good catch.

    CompositeCORFinder is the only estimator not inheriting from CORFinderBase (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 parameter

    Edited by Pierre Paleo
  • Pierre Paleo resolved all threads

    resolved all threads

  • Pierre Paleo mentioned in commit 23b49f12

    mentioned in commit 23b49f12

  • merged

  • Pierre Paleo picked the changes into the branch cherry-pick-23b49f12 with commit 782dd4d9

    picked the changes into the branch cherry-pick-23b49f12 with commit 782dd4d9

  • Pierre Paleo mentioned in commit 782dd4d9

    mentioned in commit 782dd4d9

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading