Extending ZeldovichPancake to include DM particles

#435 Merged at 98b7c03
Repository
jwise77
Branch
week-of-code
Repository
enzo
Branch
week-of-code
Author
  1. John Wise
Reviewers
Description
  • Adding support for DM particles in the ZeldovichPancake problem
  • Fixing bugs for particle movement in 1D and 2D simulations
  • Adding option for bulk gas velocity in the ZeldovichPancake problem. Added documentation for initial B-field, which was missing.
  • Adding new ZeldovichPancake_Streaming test problem with DM particles and a bulk gas velocity.
  • Commit status

Comments (6)

  1. Greg Bryan

    This looks good to me. Merging should wait until testing can be done; it will also require an update of the gold-standard (but this shouldn’t be done until PR#434 and associated gold-standard update is done).

    1. Britton Smith

      Since we’re all probably running low on build minutes, I merged this PR with tip in my own fork and ran the tests there and they’ve all passed. You can see the build here.

      1. Greg Bryan

        Thanks Britton!

        I agree, this is ready to go. Once merged, the gold-standard should be updated.

  2. John Wise author

    The tests are failing because PR 434 hasn’t been merged. I will do this and restart the tests.