[vtk-developers] Code review request: biquadratic quadinterpolation
jcplatt at dsl.pipex.com
Thu Jan 23 18:21:40 EST 2014
I tested this change using 5.10 and it did not affect colour mapping,
cutting or clipping. Unfortunately, I don't do any probing so I cannot check
There would appear to be a dependency on the starting point for the linear
quads with the 'adjust parametric coordinates' section in
EvaluatePosition(). Here, the parametric coordinates from the linear quad
are converted to parametric coordinates in the quadratic quad. The origins
of each linear quad are as follows ...
( 0, 0 ) - point 0
( 0.5, 0 ) 4
( 0.5, 0.5 ) 8
( 0, 0.5 ) 7
So I would say that the proposed change is consistent with these origins.
vtkQuadraticQuad uses the same linear quads as the old vtkBiQuadraticQuad so
it could be a problem here also.
----- Original Message -----
From: "Marcus D. Hanwell" <marcus.hanwell at kitware.com>
To: "David Thompson" <david.thompson at kitware.com>
Cc: "VTK Developers" <vtk-developers at vtk.org>
Sent: Thursday, January 23, 2014 9:16 PM
Subject: Re: [vtk-developers] Code review request: biquadratic
> On Thu, Jan 23, 2014 at 3:48 PM, David Thompson
> <david.thompson at kitware.com> wrote:
>> Hi Marcus,
>>> There is a topic on Gerrit I am having trouble finding a reviewer for,
>> Soeren Gebbert (soerengebbert at googlemail.com) is the original developer.
>> I'm really swamped right now, but it would be great to verify with Sandia
>> that the change does not mess with the Exodus reader's numbering of nodes
>> within the cell... several changes have done so in the past.
> It would be nice to figure out what the expected behavior is too,
> because this doesn't break any tests I can see. It is a single line
> change, I can email the original developer but don't have much more
> time I can put into this.
> Powered by www.kitware.com
> Visit other Kitware open-source projects at
> Follow this link to subscribe/unsubscribe:
More information about the vtk-developers