Skip to content
GitLab
  • Menu
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 61
    • Issues 61
    • 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
  • !470

Gizmo rewrite and bug fixes

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Bert Vandenbroucke requested to merge gizmo into master Dec 15, 2017
  • Overview 3
  • Commits 3
  • Changes 9

Rewrote Gizmo so that it no longer needs the particle time step during the flux computation. This breaks manifest conservation of mass, momentum and energy, but makes the scheme more similar to the other hydro schemes. Got rid of a lot of unused particle variables. Fixed a bug in the way the velocity for inactive particles was set in the flux computation, and a bug in the way the interface area was calculated. The square test, which was broken, now successfully runs again.

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: gizmo