I'll check...

--
Barnaby Rowe
Postdoctoral Research Associate

Department of Physics & Astronomy
University College London
Gower Street
London WC1E 6BT
United Kingdom

On 18 Mar 2013, at 13:53, Mike Jarvis wrote:

> In galsim/catalog.py:
> 
> >  
> > -    def nObjects(self):
> > -        """Return the number of objects in the catalog
> > -        """
> > -        return self.nobjects
> > +        # TODO: Is there a faster way to do this if all we care about is nobjects?
> > +        self.nobjects = self.data.shape[0]  
> 
> Thanks for this guys. I made the change to a combination of the two "dumb" algorithms, since they are "correct by inspection". I doubt the marginal speedup of going to the buffered case is really needed, and I'm a bit concerned about the edge case issues that Melanie pointed out, so I didn't want to risk it. However, I did add a comment in the main code about the timing script, so if ever people want to revisit the issue, they'll know where to go for that.
> 
> —
> Reply to this email directly or view it on GitHub.
>