Re: [Numpy-discussion] Suggestion for recarray.view

2008-09-19 Thread Stéfan van der Walt
2008/9/19 Travis E. Oliphant [EMAIL PROTECTED]:
 #---
 def view(self, dtype=None, type=None):
 if dtype is None:
 return ndarray.view(self, type)
 elif type is None:
 try:
 if issubclass(dtype, ndarray):
 return ndarray.view(self, dtype)
 except TypeError:
 pass
 dtype = sb.dtype(dtype)
 if dtype.fields is None:
 return self.__array__().view(dtype)
 return ndarray.view(self, dtype)
 else:
 return ndarray.view(self, dtype, type)
 #---

 This looks pretty good to me.

 +1 for adding it.

+1 and another +1 to your karma for requesting peer review.  Let me
know if you need me to whip up a couple of tests for verifying the
different usage cases.

Cheers
Stéfan
___
Numpy-discussion mailing list
Numpy-discussion@scipy.org
http://projects.scipy.org/mailman/listinfo/numpy-discussion


Re: [Numpy-discussion] Suggestion for recarray.view

2008-09-19 Thread Pierre GM
On Friday 19 September 2008 04:13:39 Stéfan van der Walt wrote:

 +1 and another +1 to your karma for requesting peer review.  Let me
 know if you need me to whip up a couple of tests for verifying the
 different usage cases.

That'd be lovely. I'm a bit swamped with tricky issues in mrecords and 
dependents...
___
Numpy-discussion mailing list
Numpy-discussion@scipy.org
http://projects.scipy.org/mailman/listinfo/numpy-discussion


[Numpy-discussion] Suggestion for recarray.view

2008-09-18 Thread Pierre GM
All,
I'd like to submit the following suggestion for recarray.view, so that it 
could accept two keywords like standard ndarrays do. 
As a change in records.py can potentially affect a lot of people (probably 
more than a change in numpy.ma), I'm not confident enough to commit it.
Consider that as an attempt of having it peer-reviewed before inclusion. 
Cheers
P.
#---
def view(self, dtype=None, type=None):
if dtype is None:
return ndarray.view(self, type)
elif type is None:
try:
if issubclass(dtype, ndarray):
return ndarray.view(self, dtype)
except TypeError:
pass
dtype = sb.dtype(dtype)
if dtype.fields is None:
return self.__array__().view(dtype)
return ndarray.view(self, dtype)
else:
return ndarray.view(self, dtype, type)
#---
___
Numpy-discussion mailing list
Numpy-discussion@scipy.org
http://projects.scipy.org/mailman/listinfo/numpy-discussion


Re: [Numpy-discussion] Suggestion for recarray.view

2008-09-18 Thread Travis E. Oliphant
Pierre GM wrote:
 All,
 I'd like to submit the following suggestion for recarray.view, so that it 
 could accept two keywords like standard ndarrays do. 
 As a change in records.py can potentially affect a lot of people (probably 
 more than a change in numpy.ma), I'm not confident enough to commit it.
 Consider that as an attempt of having it peer-reviewed before inclusion. 
 Cheers
 P.
 #---
 def view(self, dtype=None, type=None):
 if dtype is None:
 return ndarray.view(self, type)
 elif type is None:
 try:
 if issubclass(dtype, ndarray):
 return ndarray.view(self, dtype)
 except TypeError:
 pass
 dtype = sb.dtype(dtype)
 if dtype.fields is None:
 return self.__array__().view(dtype)
 return ndarray.view(self, dtype)
 else:
 return ndarray.view(self, dtype, type)
 #---
   
This looks pretty good to me.

+1 for adding it.

-Travis

___
Numpy-discussion mailing list
Numpy-discussion@scipy.org
http://projects.scipy.org/mailman/listinfo/numpy-discussion