On Mar 17, 2013, at 7:01 PM, Mike Jarvis <notifications@github.com> wrote:

> In galsim/lensing.py:
> 
> >              else:
> >                  g1.append(sbii_g1.xValue(pos+self.offset))
> >                  g2.append(sbii_g2.xValue(pos+self.offset))
> > +                if get_kappa:
> > +                    kappa.append(sbii_kappa.xValue(pos+self.offset))
> > +                else:
> > +                    kappa.append(0.)
> Correction. In parallel with NFWHalo, it should be getConvergence, not getKappa. And on a related note, I think getMagnification is fine, instead of getMag (or getMu, which would be another option). Clarity is more important that brevity for maintainable code.
> 
Do you think it makes sense to have both getConvergence and getMagnification?
(And I'll change the NFWHalo one from getMag to getMagnification as well.)