I added a small routine that adds stellar wind from a single star.

#388 Merged at 787a665
Repository
enzo-yuan
Branch
week-of-code
Repository
enzo-dev
Branch
week-of-code
Author
  1. Yuan Li
Reviewers
Description
  • added wind option to CollapseTest (inflow boundary with fixed velocity)

  • Add Stellar Wind from an AGB star (e.g. Mira)

  • make changes so that ShockInABoxInitialize can use Zeus method.

Comments (6)

  1. Greg Bryan

    This looks good to me -- a few minor comments below but I approve! The other thing to do is to add some documentation.

  2. Yuan Li author

    Thank you, Greg! I have fixed the problems and added doc (though not sure where to put the stellar wind part).

  3. Brian O'Shea

    This also looks good to me - I'll run the test suite and report back, but I think we can merge it in pretty soon. (And sorry for the delay on this!)

  4. Philipp Grete

    Looks good to me as well (apart from the three small/style comments below, which might be worth looking at before the final merge).

  5. Brian O'Shea

    The code still passes the test suite following @pgrete 's suggested changes, and I think we're ready to merge. I will do so momentarily. Thanks to @pgrete and @gbryan for reviewing!