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 188
    • Issues 188
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 37
    • Merge requests 37
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • 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
  • !2685

Merged
Created Jan 26, 2022 by Manish Mathai@mmathaiDeveloper

Fix out of bounds cell location searchs in VolumeRendererStructured

  • Overview 22
  • Commits 1
  • Pipelines 6
  • Changes 1

RectilinearLocator::LocateCell and UniformLocator::LocateCell assume that the point is inside the bounds of the cell set but may return a cell location that does not exist, i.e. cell location that is outside of the bounds of the cell set, due to errors from floating-point operations.

This fix, given the assumption above, bounds the calculated cell location to within the cell set by capping the indices to within the limits. This fixes the cause of the crash in Ascent that is noted at https://github.com/Alpine-DAV/ascent/issues/825

Backport: release

Edited Jan 26, 2022 by Vicente Bolea
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: volume-renderer-oob-fix