Add min as selectable aggregation to maxmem diff summary#2776
Conversation
…mary and resource diff summary
|
A new Pull Request was created by @gartung for branch master. @akritkbehera, @cmsbuild, @iarspider, @raoatifshad, @smuzaffar can you please review it and eventually sign? Thanks. |
|
cms-bot internal usage |
|
enable profiling |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-509dca/53633/summary.html Comparison SummarySummary:
|
|
The behavior of "min" and "max" look ok. The "max of abs" is a bit weird because it changes the negatives to positives, which is exactly as requested, but is not very useful. Would be possible to show the original (positive or negative) values in the tables, and do the sorting based on "max of abs"? If that gets too weird, we could drop the "max of abs". The "min" is already an improvement. |
…d for posittive and green for negative.
|
Pull request #2776 was updated. |
|
Pull request #2776 was updated. |
|
Pull request #2776 was updated. |
|
Pull request #2776 was updated. |
|
please test |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-509dca/53662/summary.html Comparison SummarySummary:
|
|
Looks reasonable |
No description provided.