Allow detailed physics profile output to be switched off#4416
Allow detailed physics profile output to be switched off#4416timothy-nunn wants to merge 1 commit into
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4416 +/- ##
==========================================
+ Coverage 48.82% 48.84% +0.01%
==========================================
Files 151 151
Lines 29409 29413 +4
==========================================
+ Hits 14360 14366 +6
+ Misses 15049 15047 -2 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
@timothy-nunn May as well change the value of |
I would probably opt to do that in another PR since it will cause massive regression failures |
|
I dont see the need for this option being exposed on the main process cli, especially as the MFILE is explicitly the machine readable version. I prefer the option of reducing the number of points recorded. This is surely only talking about 121kB |
Adds an option to remove detailed physics profiles from the MFile.
process -i my.IN.DAT --no-profilesI have tested the plotting of a summary document from such an MFile and no errors occur however the figures for the detailed physics profiles contain no data/ a zero line.

I think this is fine since the user explicitly requested no profiles, however I am happy to discuss skipping these plots for a reduced MFile if we think that is worthwhile.