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

Fix support for enabling 64bit vtkm::Id and double precision

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Will Usher requested to merge Twinklebear/vtk-m:fix-64bit-and-doubles into master Jul 22, 2015
  • Overview 5
  • Commits 1
  • Changes

The CMake flag and define set in Configure.h.in differ in their capitalization of the 'm' in VTKm so I've changed the CMake flags to match those used in Configure.h.in. I'm not sure if fixing it the other way around might be preferred to keep all the CMake flags prefixed with VTKm instead of mixing VTKm/VTKM.

This fix also revealed some methods in CellSet and CellSetExplicit where ints where used instead of vtkm::Ids which caused errors with child classes who overrided methods and returned a vtkm::Id instead of an int.

Also fixed issues that appeared in TestOutOfMemory which got out of date due to not being compiled since the VTKM_USE_64BIT_IDS flag would never be set. The test now runs and passes when 64bit ids are enabled.

Let me know what you think, I wasn't sure which way to change the flag/define names to match convention.

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: fix-64bit-and-doubles