lsst-ts / ts_phosim

High-Level Module to Perturb the PhoSim
GNU General Public License v3.0
0 stars 1 forks source link

tickets/DM-37062: moved sqrt(2) inside rms calculation. #76

Closed jfcrenshaw closed 1 year ago

jfcrenshaw commented 1 year ago

The previous implementation of the zernike conversion had an empirical factor of 1/sqrt(2). Josh realized this was because there was supposed to be a division by 2 inside the square root for the rms calculation. I moved this factor inside the square root and updated the documentation to reflect this change.

This change is purely semantic -- it has no impact on any calculations.