Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • SWIFTsim SWIFTsim
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 52
    • Issues 52
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 17
    • Merge requests 17
  • Deployments
    • Deployments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • SWIFT
  • SWIFTsimSWIFTsim
  • Merge requests
  • !739

Non periodic partitioning

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Peter W. Draper requested to merge non-periodic-repart into master Feb 15, 2019
  • Overview 9
  • Commits 13
  • Changes 3

The graph of cells that we currently use assumes that the space wraps at the boundaries. This is not true for non-periodic volumes and we have no work across those boundaries. These updates allow for that.

They also allow for vertex weights of zero, as non-periodic spaces can have many empty cells, which should not be considered to carry any work. This latter point is important as this allows empty cells to be gathered to a single node (more frequently anyway, having uniform weights seems not to disallow this effect), which reduces the area of the regions, reducing the number of proxies required (important for spaces with 256x256x256 cells, with a small number of active cells).

Fixes #540 (closed).

Edited Apr 23, 2019 by Peter W. Draper
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: non-periodic-repart