[vtk-developers] Crossing library boundaries

[Bill Lorensen]

I agree with David G. The code is ConeSource is straight forward.
Perhaps we should add a set direction to ArrowSource and possibly
other sources. Maybe a helper class to compute the transform,

Bill

On Wed, Sep 15, 2010 at 2:19 PM, David Gobbi <david.gobbi at gmail.com> wrote:
> On Wed, Sep 15, 2010 at 12:04 PM, David Doria <daviddoria+vtk at gmail.com> wrote:
>>
>> Jim, if David Gobbi is right and we can move vtkLandmarkTransform to Common
>> then it would be fine to allow vtkArrowSource to accept a Point1 and Point2,
>> right?
>
> If you need to generate a transform from two points, then you
> shouldn't be use vtkLandmarkTransform.   There are far more efficient
> ways of doing it.  See the code in vtkConeSource for setting the
> Direction of the cone, for instance.  I recommend that you write code
> similar to that, instead of using vtkLandmarkTransform.
>
>> David G, if we don't hear any screaming would you volunteer to verify my
>> commit of moving vtkLandmarkTransform to Common (to github) before we push
>> to VTK master?
>
> Well, I'm still not sure if it belongs in Common or if it belongs in
> Filtering.  I only want it to be moved once.
>
>  - David
>