Evan Bauer

Yes, but it looks like Bill deleted it from the `other_physics_hooks` test case shortly after the github transition. Anne's example seems to suggest this is an important control, so I...

Yes, I think this makes sense. I think the difference mostly comes down to the inclusion of `smooth_brunt_B` in brunt.f90, so just be aware that this has had some smoothing...

I don't think this issue should be closed yet? The comment in controls.defaults is good enough for going ahead with a release, but I think there's still an underlying issue...

Personally, I feel like a "fix" like `secyear` will just get us into [bikeshedding](https://en.wiktionary.org/wiki/bikeshedding) territory, so I'd prefer just to leave it. (But I fully approve of `secday`.)

I think it would be better for us to adopt a practice of avoiding commits with many lines of superficial whitespace changes. Can you configure your editor to not auto-format...

It looks like `set_new_abundances` is not a flag, but a subroutine in star/private/adjust_xyz.f90 that gets called to fix things up after net changes. There's a potentially useful comment buried deep...

Thanks for taking this on. Just pushed an update to `do1_test_source` so that it will test the relevant cases for bit-for-bit issues.

I ended up having to tweak `1.4_ms_op_mono` so that the photo restart would give something meaningful. It was giving a superficial failure at first because it was set to `max_model_number...

This is mostly done on the [thermohaline hackathon](https://github.com/MESAHub/mesa/tree/thermohaline_hackathon) branch. We just need to promote the hard-coded magnetic field value to an optional input parameter for the thermohaline calls, and implement...

Thanks for this thorough report! I'm interested in taking a closer look, but haven't had a chance to dig into the details yet. One quick question on a point I...