Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • VTK-m VTK-m
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 206
    • Issues 206
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 42
    • Merge requests 42
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Artifacts
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Terraform modules
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • VTKVTK
  • VTK-mVTK-m
  • Merge requests
  • !98

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

  • Review changes

  • Download
  • 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
  • Pipelines 0
  • 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
Reviewers
Request review from
Time tracking
Source branch: fix-64bit-and-doubles