On Sat, Dec 5, 2009 at 12:12 AM, Travis Oliphant <oliph...@enthought.com>wrote:
> > On Dec 4, 2009, at 10:09 AM, Charles R Harris wrote: > > > > On Fri, Dec 4, 2009 at 6:29 AM, Travis Oliphant <oliph...@enthought.com>wrote: > >> >> What do people think of applying patch #1085. This patch makes a copy of >> inputs when the input and output views overlap in ways in which one >> computation will change later computations. >> >> > I'd rename the function views_share_base_data, make the loop to find the > base an inplace function, and break the long lines in the test. > > It looks like the routine doesn't try to determine if the views actually > overlap, just if they might potentially share data. Is that correct? That > seems safe and if the time isn't much it might be a nice safety catch. But > it shouldn't go into 1.4. > > > It sounds like with a little clean-up that it can be applied to the trunk. > I agree that it shouldn't go into 1.4. > > Agreed, it looks to add a bit of safety. Chuck
_______________________________________________ NumPy-Discussion mailing list NumPy-Discussion@scipy.org http://mail.scipy.org/mailman/listinfo/numpy-discussion