Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Flux scaling error fix #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Flux scaling error fix #12

wants to merge 1 commit into from

Commits on Feb 23, 2016

  1. Flux scaling erro fix 

    I tracked down an error in the flux calibration stage in some data I
have been working with. If there was a nod and shuffle source observed,
then the NEXP, SEXP, NSUBEXPS header fields will still contain this
information when other observing modes are used. When the pipeline uses
the ’ns’:True parameter, signifying that N&S has been used, the exptime
is incorrectly re-calculated using these N&S parameters for exposures
made with other modes - like ClassicEquall for the standard star. This
can obviously throw off the calibration as the observed flux is scaled
in by exptime.

The fix checks the observing mode of a frame. If it is not a N&S, then
falls back to the exptime in original header, which has not been edited.
    Tim Galvin committed Feb 23, 2016
    Configuration menu
    Copy the full SHA
    226a17c View commit details
    Browse the repository at this point in the history