Done.

On Mar 21, 2013, at 10:15 PM, Mike Jarvis <notifications@github.com> wrote:

> In galsim/base.py:
> 
> > +        galsim.NFWHalo classes, which compute shears according to some lensing power spectrum or
> > +        lensing by an NFW dark matter halo.
> > +
> > +        After this call, the caller's type will be a GSObject.
> > +        This means that if the caller was a derived type that had extra methods beyond
> > +        those defined in GSObject (e.g. getSigma() for a Gaussian), then these methods
> > +        are no longer available.
> > +
> > +        @param g1      First component of lensing (reduced) shear to apply to the object.
> > +        @param g2      Second component of lensing (reduced) shear to apply to the object.
> > +        @param mu      Lensing magnification to apply to the object.
> > +        """
> > +        shear = galsim.Shear(g1=g1, g2=g2)
> > +        scale = np.sqrt(mu)
> > +        self.SBProfile.applyShear(shear._shear)
> > +        self.SBProfile.applyTransformation(galsim.Ellipse(np.log(scale))._ellipse)
> Please call applyMagnification, rather than use Ellipse explicitly with applyTransformation. The fact that it is done this way is an implementation detail, and as such, it should only appear in one location. Everywhere else should use applyMagnification to apply a magnification.
> 
> —
> Reply to this email directly or view it on GitHub.
> 


-------------------------------

Rachel Mandelbaum
rmandelb@andrew.cmu.edu
http://www.andrew.cmu.edu/~rmandelb/