search

LSSTDESC / descwl-shear-sims

simple simulations for testing weak lensing shear measurement
BSD 3-Clause "New" or "Revised" License
10 stars 9 forks source link

make galsim draw method optional, default auto #153

Closed esheldon closed 3 years ago

esheldon commented 3 years ago

I think just turning on 'phot' is correct because wldeblend gives flux in electrons

However, it can use a ton of memory so probably want to put a cap on number of photons

esheldon commented 3 years ago

This gets pretty slow in some cases if there are objects with a lot of photons.

Because we are drawing all objects at once (other than BRIGHT) we can't control this by using fewer photons for some objects.

esheldon commented 3 years ago

I think we would need to implement #117 to make this faster

esheldon commented 3 years ago

@rmjarvis does not think we can speed up the photon shooting, making it optional and we can do targeted studies

esheldon commented 3 years ago

@beckermr this one should be uncontroversial