Re: [IMP-dev] [IMP-commits] r1297 - trunk/modules/algebra/include

[Keren Lasker <>]

then maybe I should update that for Vector3D and Rotation3D as well,  
as it was not possible accessing Transformation3D which was  
constructed from the default one before.
On Jan 28, 2009, at 7:44 AM, Daniel Russel wrote:

> It also means that when someone creates a default Transformation3D,  
> you don't know if they want the identity transform or just will  
> initialize it later. And it's not symmetric with the Vector3D and  
> the Rotation3D.
>
>
> On Jan 27, 2009, at 10:16 PM, Notification of IMP commits wrote:
>
> > Author: 
> > Date: 2009-01-27 22:16:23 -0800 (Tue, 27 Jan 2009)
> > New Revision: 1297
> >
> > Modified:
> >  trunk/modules/algebra/include/Transformation3D.h
> > Log:
> > change to default constrator to initilize the identity transformation
> >
> > Modified: trunk/modules/algebra/include/Transformation3D.h
> > ===================================================================
> > --- trunk/modules/algebra/include/Transformation3D.h	2009-01-27  
> > 22:26:43 UTC (rev 1296)
> > +++ trunk/modules/algebra/include/Transformation3D.h	2009-01-28  
> > 06:16:23 UTC (rev 1297)
> > @@ -25,6 +25,8 @@
> >  typedef Transformation3D This;
> >  //! construct and invalid transformation
> >  Transformation3D(){
> > +    trans_=Vector3D(0.0,0.0,0.0);
> > +    rot_ = identity_rotation();
> >  }
> >  /** basic constructor*/
> >  Transformation3D(const Rotation3D& r, const Vector3D& t)
> >
> > _______________________________________________
> > IMP-commits mailing list
> >
> > https://salilab.org/mailman/listinfo/imp-commits
>
> _______________________________________________
> IMP-dev mailing list
>
> https://salilab.org/mailman/listinfo/imp-dev