search

PaulHancock / Aegean

The Aegean source finding program and associated tools
http://aegeantools.rtfd.io/
Other
47 stars 14 forks source link

Island fluxes do not match integrated fluxes #13

Closed nhurleywalker closed 9 years ago

nhurleywalker commented 9 years ago

I run Aegean with the --island option and cross-match the resulting _isle and _comp tables with an exact match on the island column. When I plot the integrated flux as derived from the component fit against the integrated flux (divided by eta) as derived from the island floodfill, the results are out by a factor of three. Seedclip is 8 and floodclip is 5, so at the faint end it might be somewhat understandable (although eta should fix that) but it doesn't make a lot of sense at the high end. Linear and log plots attached. island_v_int_linear island_v_int_log

PaulHancock commented 9 years ago

This has been fixed in the master branch as of 2/10/15.

rmalexan commented 8 years ago

I cloned the repo of Aegean today (just to check) and ran a similar test to find what appears to be the same mismatch between the component and island integrated flux (again matching on the island column) -- see attached. Is there a flag to turn on or something else I should be aware of?
image

PaulHancock commented 8 years ago

rmalexan,

Despite my previous posts, it appears that the solved bug has been reintroduced to the master branch of Aegean. The good news is that the current development branch has fixed this issue.

It seems that the issue is related to the integrated island flux that is calculated by Aegean. On the 1904-66_SIN.fits test image, the current master branch gives an island flux that is about 15% of what I would expect. The current dev branch gives the expected flux, and the island int/eta calculation gives a 1:1 match to the component integrated flux..

I plan to push the changes in the dev branch onto the master branch next week. You can either use the dev branch now or wait for me to push the changes to the master branch.

Paul.