Add min max to post processing script#237
Add min max to post processing script#237Benjamin Went (MetBenjaminWent) wants to merge 10 commits intoMetOffice:mainfrom
Conversation
|
This change has illustrated that the existence of the |
|
it might be useful to order the merge management and put #242 ahead of this one. |
That seems sensible to me! I'd paused this PR temporarily whilst I had a think about the previous suggestions, but we are thinking of pausing those here right Ed Hone (@EdHone)? |
…/Vernier into add_min_max_to_post_proc
Description
To effectively assess runtime properties of an application using Vernier, we need to have access to the self min and the max times of a caliper.
I've updated the new class based post-processing script with this functionality, which I believe to be a baseline feature, as opposed to an optional feature uses can add with their own scripts outside of Vernier.
The python testing has been updated as required by the script.
Blockers
#242
Linked issues
Closes #238
Type of change
How has this been tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist: