@aemerick and @cdb09f can you review this PR?
It looks good. The one question I have is, what is the reason for removing the check for a successful return from InitializeTemperatureFieldForH2Shield, and the ENZO_FAIL error?
The ENZO_FAIL is a relic from a time when we didn’t have error exceptions that produced backtraces. There are still some in the code, but I try to remove them everytime I see one in a change.
Happy to. Probably within the next few days.
We have four approvals, comments all addressed, passes tests – I’m merging. Thanks all!