lsst-sims / legacy_sims_GalSimInterface

code to seamlessly generate simulated images using GalSim based off of simulated catalogs generated by CatSim
5 stars 7 forks source link

Add DCR to the surface operations list #53

Closed cwwalter closed 6 years ago

cwwalter commented 6 years ago

When using the atmospheric PSF (or in all cases?) we should add DCR to the list of surface operations. @rmjarvis can you comment on any issues to be aware of?

rmjarvis commented 6 years ago

Sorry. Should have commented here. Cross-reference

danielsf commented 6 years ago

Is there something I can read to get up to speed on what is meant by "surface operations"? I started seeing that term being used about six months ago. I assume it is a GalSim term of art. Thanks.

rmjarvis commented 6 years ago

Yes, it is a term of art. Basically, they are functions that act on the photon array in between where they are shot (rendering the surface brightness profile at the surface of the silicon) and accumulated in the sensor.

We've been using them to

It also provides a hook for not-yet-implemented effects, such as

There's unfortunately not great documentation of them currently. Our docs (as everyone's) still needs some work. And this is one area where the doc strings probably aren't sufficient to get a good feel for how to use them. Perhaps the photon array test suite would be a place to look for semi-real-world usage.

cwwalter commented 6 years ago

This was done a long time ago..