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.

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

Verify apcor parameters being passed to photometry calculations #65

Closed
drusk opened this issue Jul 3, 2013 · 1 comment · Fixed by #70
Closed

Verify apcor parameters being passed to photometry calculations #65

drusk opened this issue Jul 3, 2013 · 1 comment · Fixed by #70

Comments

@drusk
Copy link
Contributor

drusk commented Jul 3, 2013

@ijiraq I found a note to myself to check with you about the apcor parameters being passed to daophot.py. Maybe you can take a quick look, specifically at the swidth property of the ApcorData class in https://github.com/drusk/MOP/blob/master/src/ossos-pipeline/ossos/gui/image.py

I did it this way based on our e-mail conversations while you were away, in which you said apcor files have the following format:

ap_in ap_out apcor apcor_err
numbers like
4 10 0.4 0.05
are normal.

ap == ap_in
insky == ap_out+1
outsky == insky + ap_in

You then also said:

swidth = outsky - insky (if you look through the dophot.pl script there is likely a line someplace that says: outsky = ousky - insky ... poor choice of variable names).

Therefore I calculated that swidth = insky + ap_in - insky = ap_in.
But then swidth == aperature ❓ (both get passed to daophot)

Did I misinterpret something? If you can do a quick review of this it would be good.

@ijiraq
Copy link
Member

ijiraq commented Jul 4, 2013

nominally swidth = aperture , that is correct. But not required for dophot.pl to return a correct result.

swidth can be selected as a different value than aperture and a valid value will be measured. swidth=aperture is a handy rule-of-thumb for doing photometry which works for our typical fields and source type. Your code looks correct and calls to phot need to pass both values, ossos pipeline just chooses to use the same value for both.

Keeping the 'swidth' and 'aperture' as separate attributes is a good design choice as the choice of making them equal could be changed during evaluation of the pipelines performance.

JJ

On 2013-07-03, at 3:30 PM, David Rusk wrote:

@ijiraq I found a note to myself to check with you about the apcor parameters being passed to daophot.py. Maybe you can take a quick look, specifically at the swidth property of the ApcorData class in https://github.com/drusk/MOP/blob/master/src/ossos-pipeline/ossos/gui/image.py

I did it this way based on our e-mail conversations while you were away, in which you said apcor files have the following format:

ap_in ap_out apcor apcor_err
numbers like
4 10 0.4 0.05
are normal.

ap == ap_in
insky == ap_out+1
outsky == insky + ap_in

You then also said:

swidth = outsky - insky (if you look through the dophot.pl script there is likely a line someplace that says: outsky = ousky - insky ... poor choice of variable names).

Therefore I calculated that swidth = insky + ap_in - insky = ap_in.

But then swidth == aperature (both get passed to daophot)

Did I misinterpret something? If you can do a quick review of this it would be good.


Reply to this email directly or view it on GitHub.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants