idaholab / moose

Multiphysics Object Oriented Simulation Environment
https://www.mooseframework.org
GNU Lesser General Public License v2.1
1.67k stars 1.03k forks source link

Timed print in multiapp seems not having the leading app name #15051

Open YaqiWang opened 4 years ago

YaqiWang commented 4 years ago

Reason

I'v noted this in one of my calculations:

wrapped_app: Assembling the matrix for update .........................................................
..........................................................................................
...............                                                                            [163.52 s]

Design

Somehow the console stream is not handling \n properly in multiapp.

Impact

Minor for prettifying the console outputs.

YaqiWang commented 4 years ago

For some reason, looks like Outlier Variable Residual Norms: is also not prefixed.

permcody commented 4 years ago

I'm not at all surprised. TimedPrint isn't buffered, it can't be because you want real-time updates to the screen. The console outputter is completely buffered so that we can do things like indent and colorize the text. Solving this one will be pretty challenging!

GiudGiud commented 9 months ago

Need a MWE to fix this. I fixed a bunch of perf graph issues earlier this year, there's a non-zero chance i fixed that one