[vtkusers] vtkCutter leaves "gaps" -- [SOLVED] : sortof

[David Gobbi]

Hi Gerrick,

I'm a bit out of my "element" here, since I don't do any FEM work,
myself.  If you submit this change to VTK's patch review system
(see http://www.vtk.org/Wiki/VTK/Git/Develop) then it might attract
attention from the folks who can help you get it into VTK.

  David

On Fri, Jan 17, 2014 at 5:38 AM, Gerrick Bivins
<Gerrick.Bivins at halliburton.com> wrote:
> David,
>
> For the method where I saw the problem:
>
> vtkPolyhedron::CheckContourDimensions
>
>
>
> eigenvalues were stored as doubles.
>
> Then the ratio of the eigenvalues was failing the tolerance check.
>
> Are you saying that setting this tolerance based on the precision of the
> points
>
> would change this?
>
>
>
> Gerrick
>
> From: David Gobbi [mailto:david.gobbi at gmail.com]
> Sent: Wednesday, January 15, 2014 10:43 AM
> To: Gerrick Bivins
> Cc: vtkusers at vtk.org; Jeff Lee
> Subject: Re: [vtkusers] vtkCutter leaves "gaps" -- [SOLVED] : sortof
>
>
>
> Hi Gerrick,
>
>
>
> It would be nice to make the tolerance into an option that could be set
> somewhere, but I don't know where the "Set" method for this option could go.
>
>
>
> Are there any other VTK developers who can offer some advice?  Maybe use a
> tolerance of 1e-3 for single-precision points and a tolerance of 1e-12 for
> double-precision points?  That would seem reasonable to me.
>
>
>
>   David
>
>
>
> On Mon, Jan 13, 2014 at 3:53 PM, Gerrick Bivins
> <Gerrick.Bivins at halliburton.com> wrote:
>
> So I spent a couple of days trying to narrow this down.
>
> It turns out to be a tolerance issue in a method of vtkPolyhedron that
> attempts to calculate the “CheckContourDimensions”.
>
> The tolerance is hardcoded to something like .001 and the check ends up
> returning a dimension of “1” for
>
> the cells that had “gaps”.
>
> Decreasing the tolerance to 1e-12 allows the vtkCutter to work for all the
> models that I’ve tried so far.
>
>
>
> I updated the bug with the info I found.
>
>
>
> Any chance of getting a fix in for this in :
>
> 5.10.* patch?
>
> 6.1 release?
>
>
>
> Gerrick
>
> ________________________________
> This e-mail, including any attached files, may contain confidential and
> privileged information for the sole use of the intended recipient. Any
> review, use, distribution, or disclosure by others is strictly prohibited.
> If you are not the intended recipient (or authorized to receive information
> for the intended recipient), please contact the sender by reply e-mail and
> delete all copies of this message.