-
Notifications
You must be signed in to change notification settings - Fork 2
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
Use galsim
as realistic source galaxy generator
#48
base: master
Are you sure you want to change the base?
Conversation
Hi @aymgal, thank you for proposing this feature -- I'm very much on board with the idea of using GalSim for realistic sources. From what I can gather from a first-pass scan, your implementation interfaces with the prior and config classes in just the way I intended. (Sorry for the scant documentation with the config files!) I don't have particular objections to using
Once again, thank you for working on this very helpful feature. Looking forward to discussing this more soon :) |
Happy that the implementation is ok on your side! Got it for the noise generation, indeed I'm learning that it's best practice to introduce noise in an online fashion. I'm currently on "stand by" for the development of this pull request, but feel free to merge it if you would like to use / improve it. I will likely be doing some adjustments before the end of the month though. Finally, what do you mean by |
For some reason, I haven't been making significant changes to Baobab as I'm finishing up my other project, so this PR won't diverge too much. I can wait until whenever you get a chance to work on it. |
Hi!
I am currently playing with your very useful code to generate a training set for a project, and have been working on implementing realistic source into your pipeline.
The way I implemented it can be summarised as follows :
'GALSIM'
source profile. It can easily be generalised to other realistic galaxy generators. New sampled parameters has been defined accordingly.'INTERPOL'
profile of lenstronomy, with the best resolution possible (depending onsupersampling_factor
), to avoid at maximum accuracies with the interpolation. This requires my last lenstronomy PR to be merged in your lenstronomy package.'INTERPOL'
is replacing'SERSIC_ELLIPSE'
.galsim
, some additional parameters (on top of the sampled ones) should be accessible to the user. For that reason, I created a very generalcfg.external
field in the config file, where the user can specific some settings corresponding to the source/lens profiles being used.The script
generate_pixel.py
is an adaptation of the originalgenerate.py
, that supports this new prior.As a first step, this PR is only intended to start discussion about this feature that I think should be useful, and may not be merged into the master right now.
I would be very interested to hear about your remarks/comments please. Especially concerning the implementation, if this is or not on the right spirit regarding of your current implementation (especially last point above).
Thanks!
PS : Travis CI tests might failed in the beginning, because of galsim installation issues...let's see.