-
Notifications
You must be signed in to change notification settings - Fork 5
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
AutoProf fails to fit isophotes of face-on galaxies #8
Comments
btw, during initialization, the optimization of global ellipticity also involves AutoProf/autoprof/pipeline_steps/Isophote_Initialize.py Lines 285 to 308 in 8db0fd2
I have no evidence this will cause an issue. Please weigh in on this. Thanks! |
Hi @hgao-astro, Wow, this is great work! To me this seems clearly better for face-on galaxies. My only concern is for edge-on systems with ellip close to 1. In that case the shape becomes very sensitive to small changes in the ellipticity and so part of the remapping of I agree for the initialization it is fine to just keep it since its more the optimization that encounters the issue. |
Thanks for the prompt reply. Let me give it a try and see if it works as expected. btw, I forgot to mention that I used a perturbation scale = 0.1 to achieve successful face-on fits, and have not yet evaluated the impact on computation time. |
Thanks for your patience. I have been occupied by job searching these days. I will try to get to it in Jan. 2024. |
@hgao-astro Good luck with your job search! I look forward to hearing from you in the new year! |
The optimization of isophotal ellipticity fails when the initialized ellipticity is close to 0 or 1. Because the perturbation was done after converting the ellipticity to another parameter space of$[-\infty, +\infty]$ using $\epsilon \rightarrow 0$ and $\epsilon \rightarrow 1$ (corresponds to $-\infty$ and $+\infty$ in the perturbed parameter space) negligible. Since there are few galaxies with global ellipticity close to 1, the problem mostly affects nearly face-on galaxies.
_inv_x_to_eps
and then scaling back using_x_to_eps
. The functional form makes any finite perturbation atAutoProf/autoprof/pipeline_steps/Isophote_Fit.py
Lines 628 to 631 in 8db0fd2
I assume that the purpose of such scaling is to do optimization in a continuous parameter space. However, since the perturbation was done in a Monte-Carlo manner, I presume that it is not critical to do such scaling. I try directly perturbing the ellipticity and when it hits its parameter boundary I simply reflect it against the boundary.
This temporary solution seems working. Let me know if you have other ideas. I would be happy to initiate a pull request when ready. Below I show a failed example and a successful fit using the new code above.
The text was updated successfully, but these errors were encountered: