search

xjiang4 / ellipsoids

Automatically exported from code.google.com/p/ellipsoids
Other
0 stars 1 forks source link

ellipsoid.getCopy is implemented incorrectly #114

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
Currently implementation calls the constructor and doesn't copy absTol, relTol 
and other properties.

Also, current code calls ellipsoid constructor while in fact it is not 
necessary, all that is needed is calling a default constructor and then copying 
the properties directly like

ell.shape=self.shape 

etc.

We need to cove this method with enhanced tests to make sure that the 
implementation is correct

Original issue reported on code.google.com by heartofm...@gmail.com on 24 Apr 2013 at 8:45

GoogleCodeExporter commented 8 years ago

Original comment by heartofm...@gmail.com on 25 Apr 2013 at 8:50

GoogleCodeExporter commented 8 years ago

Original comment by heartofm...@gmail.com on 26 Apr 2013 at 7:12