Skip to content
GitLab
Projects Groups 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
    • Contributors
    • Graph
    • Compare
  • Issues 57
    • Issues 57
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 23
    • Merge requests 23
  • 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
  • !667

Change the task splitting criterion for gravity tasks

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Matthieu Schaller requested to merge gravity_depth2 into master Nov 06, 2018
  • Overview 23
  • Commits 13
  • Changes 13

Fixes #457 (closed).

Implements three main changes:

  • When deciding whether or not to split a gravity task use a relative depth and not an absolute depth.
  • Send the maximal local depth to the proxies when exchanging cells.
  • Add a drift_out implicit task to resolve the high number of gpart_drift -> gravity task dependencies.

The parameter file gets a new parameter to control the depth level of the gravity tasks Scheduler:cell_subdepth_diff_grav which defaults to 4. That is, a gravity task can only be split if it is 4 levels (or more) above the leaf cells in that part of the tree. The best default value of this parameter needs to be investigated.

Edited Nov 07, 2018 by Matthieu Schaller
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: gravity_depth2