E3SM icon indicating copy to clipboard operation
E3SM copied to clipboard

Homme: Some minor tweaks related to print to screen

Open bartgol opened this issue 3 years ago • 5 comments

Namely:

  • allow iulog to be settable if SCREAM is defined, rather than hard coded to 6 (stdout);
  • replaced a couple of print to screen with write(iulog, *);
  • allow to pass output stream to SimulationParams::print.

Plus, fixed compiler warning related to SimualtionParams members initialization order.

homme_integration and e3sm_developer passed on Chrysalis.

[BFB]

bartgol avatar Aug 08 '22 22:08 bartgol

homme_integration and e3sm_developer both pass on chrysalis.

bartgol avatar Aug 09 '22 19:08 bartgol

Retested on chrysalis after the last two commits, and the two testsuites still pass.

bartgol avatar Aug 11 '22 19:08 bartgol

Is this ready for merging?

ambrad avatar Aug 11 '22 19:08 ambrad

Yes, I'm no longer touching it.

bartgol avatar Aug 11 '22 19:08 bartgol

I'll merge this on Monday if next is open then.

ambrad avatar Aug 11 '22 20:08 ambrad