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 21
    • Merge requests 21
  • 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
  • !258

Sanitize smoothing lengths

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Matthieu Schaller requested to merge error_messages into master Sep 15, 2016
  • Overview 12
  • Commits 6
  • Changes 7

Here is a proposal to address problems such as #211 (closed).

At the start, after having read in the particles, constructed the top-level cells and all their progenitors we do a quick pass to sanitize the whole thing. If a top-level cell has more than 100000 particles (say) but has an h_max that prevents tasks associated with this cell from being split we do the following:

  • Compute the geometric mean and standard deviation of the smoothing lengths in that cell.
  • Limit the smoothing lengths of all particles in this cell to mean+4sigma (say).

Recall that the cells are always split so that the scheduler will be able to push down the tasks at a later point. This procedure is applied to the top-level cells only.

Does that make sense to you ?

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: error_messages