Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • VTK VTK
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 738
    • Issues 738
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 203
    • Merge requests 203
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Infrastructure Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • VTK
  • VTKVTK
  • Issues
  • #17981

Closed
Open
Created Aug 20, 2020 by Gonzalo Sáez@gonzalosaezm55Contributor

Proposal to use double precision tolerance in vtkMergeCells

Hi,

I'm using the vtkMergeCells class with unstructured grids with double precision points. In my problem, the merging process is very sensitive to the tolerance used in the vtkMergeCells object. However, as vtkMergeCells uses vtkKdTree to find duplicated points (if the tolerance != 0.0), the tolerance used to merge these points is limited to single precision values.

To ensure that the points are merged using a double precision tolerance, I need to rely on the class vtkCleanUnstructuredGrid implemented in Paraview. However, in my opinion the current implementation of vtkMergeCells could be compatible with double precision tolerance by using vtkIncrementalPointLocator instead of vtkKdTree.

If you are interested in this enhancement, I could work on a MR to ensure that the merging process of non-coincident points in vtkMergeCells is compatible with double precision tolerance.

Assignee
Assign to
Time tracking