1. The Enzo Project
  2. Untitled project
  3. enzo-dev
  4. Pull requests

Pull requests

#133 Merged at a13204a
Repository
clean-copy
Branch
week-of-code
Repository
enzo-dev
Branch
week-of-code

Smooth sphere interfaces in collapse/photon tests

Author
  1. John Wise
Reviewers
Description

I've added a couple of options to have a smooth interface between the sphere and ambient medium in the collapse and photon tests. Another added option is constant pressure, given a density profile, where the pressure in the sphere equals the pressure at the outer sphere radius. I've added the new parameters to the docs and it passed the CollapseTestNonCosmological test.

Comments (6)

  1. John Wise author

    Here's the CollapseTestNonCosmological test with the smoothing option on. The default behavior is still the same. To do this, I added the option CollapseTestSphereSmoothSurface[0] = 1. smooth surface

  2. Nathan Goldbaum

    This looks like a useful addition. My only concern is that the spherical CollapseTest is already extremely complicated, with many knobs and switches that have been added over the years and that it might be a good idea to come back and try to streamline a bit. That being said, I don't think my concern should hold up acceptance.

    1. Matt Turk

      I completely agree about both points. My hope is that we can either use yt => GDF => Enzo, or possibly implement yt-like operators and fluid operations that can do things like this. But, this by itself seems great.

    2. John Wise author

      Yes, the collapse test is becoming more and more complicated. I hope we can change things in 3.0 with yt/GDF or more initialization operators, like what Matt said.