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

headers of output photometry best fit #9

Closed
themiyan opened this issue Jan 11, 2019 · 5 comments
Closed

headers of output photometry best fit #9

themiyan opened this issue Jan 11, 2019 · 5 comments

Comments

@themiyan
Copy link

themiyan commented Jan 11, 2019

would be helpful if the header in best_fits *.input_res.fit files state the column names in the file itself or in the param file

@cschreib
Copy link
Owner

cschreib commented Jan 11, 2019

You're right, that is something that bothered me a couple times in the past and didn't bother to fix. I changed the code to now show proper column names. Please tell me if it's not clear to you what the names mean!

@themiyan
Copy link
Author

thanks, I'm going to say
fl_obs is confusing. I guess this means intrinsic flux right, so dust corrected? because technically observed flux is with dust.

@themiyan
Copy link
Author

Also I think there is an issue here (assuming what I said above is correct)
sed_xi_ion_2323 0

In the top panel, dark blue points should match with the blue plot (intrinsic sed) and the cyan ones should match (observed sed) ( or at least vice versa if the column are jumbled).
before i go any further looking into this have you checked this before?

@cschreib
Copy link
Owner

cschreib commented Jan 11, 2019

Ah, I guess this is even more unclear than I thought it might :) The first column is wavelength, the second column (fl_model) is the best fit flux, the third column (fl_obs) is the flux of the galaxy as provided in the input catalog, and the fourth column (unc_obs) is the uncertainty of that flux. So the last two columns are just repeating data from the input catalog, to make it easier to plot the fit and observations together. I guess I'll have to document that!

NB: there is currently no option to obtain the intrinsic (without dust) broadband fluxes, and I don't think I'll add this in the near future unless you give me a good reason to :P

@themiyan
Copy link
Author

aha! got it. Yeah, the latter is not necessary, what is important to see is the photometry comparison between observations and best fit model.

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

No branches or pull requests

2 participants