[vtk-developers] Convenience methods for vtkCamera

[Aashish Chaudhary]

Hi David,

On Thu, Jul 21, 2011 at 5:28 PM, David Gobbi <david.gobbi at gmail.com> wrote:
> I noticed these two new methods in vtkCamera today, and if possible, I'd
> like for them to be removed:
>
>  void SetModelTransformMatrix( double x00,  double x01,  double x02, double
> x03,
>    double x10,  double x11,  double x12,  double x13,
>    double x20,  double x21,  double x22,  double x23,
>    double x30,  double x31,  double x32,  double x33);
>
>  void SetEyeTransformMatrix( double x00,  double x01,  double x02,  double
> x03,
>    double x10,  double x11,  double x12,  double x13,
>    double x20,  double x21,  double x22,  double x23,
>    double x30,  double x31,  double x32,  double x33);
>
> Is it really so hard to create a vtkMatrix4x4 and then pass it as an
> argument?

No it is not. We added this as part of paraview - vtk stereo
integration. I will talk to Utkarsh (along with Nikhil) tomorrow to
figure out if we can remove them. Initially we didn't put them as I
didn't want to have something like this.

I would though like to hear reasoning behind your objections. Said
that I am not favoring these two functions but would like to know what
you are thinking.

> There is even a vtkMatrix4x4::DeepCopy method to make it easy to create
> a matrix from an array of doubles.

Sure.

>
> - David
>



-- 
| Aashish Chaudhary
| R&D Engineer
| Kitware Inc.
| www.kitware.com