[arvados] updated: 2.7.0-6017-g616d135e77

[git repository hosting]

Summary of changes:
 tools/crunchstat-summary/crunchstat_summary/dygraphs.js   | 11 +++++++++++
 tools/crunchstat-summary/crunchstat_summary/summarizer.py |  2 +-
 tools/crunchstat-summary/crunchstat_summary/webchart.py   |  3 +++
 3 files changed, 15 insertions(+), 1 deletion(-)

       via  616d135e77a3e81220c6194494efa718888a9c9c (commit)
      from  cfb1970c0ca562c7adddc32eb7b1ff85e410289e (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 616d135e77a3e81220c6194494efa718888a9c9c
Author: Peter Amstutz <peter.amstutz at curii.com>
Date:   Fri Feb 23 16:29:32 2024 -0500

    19744: Adjust chart format so legend doesn't cover data
    
    Adjust summarizer check for CPU < 50% instead of "nearest vCPU"
    
    Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <peter.amstutz at curii.com>

diff --git a/tools/crunchstat-summary/crunchstat_summary/dygraphs.js b/tools/crunchstat-summary/crunchstat_summary/dygraphs.js
index 0f28bc5932..07d9418966 100644
--- a/tools/crunchstat-summary/crunchstat_summary/dygraphs.js
+++ b/tools/crunchstat-summary/crunchstat_summary/dygraphs.js
@@ -69,6 +69,17 @@ window.onload = function() {
                     valueFormatter: fmt.iso,
                 },
             }
+            var div2 = document.createElement('div');
+            div2.setAttribute('style', 'width: 150px; height: 150px');
+            chart.options.labelsDiv = div2;
+            chart.options.labelsSeparateLines = true;
+
+            var div3 = document.createElement('div');
+            div3.setAttribute('style', 'display: flex; padding-bottom: 16px');
+            div3.appendChild(div);
+            div3.appendChild(div2);
+            chartDiv.appendChild(div3);
+
             charts[id] = new Dygraph(div, chart.data, chart.options);
         });
     });
diff --git a/tools/crunchstat-summary/crunchstat_summary/summarizer.py b/tools/crunchstat-summary/crunchstat_summary/summarizer.py
index e03e6bb461..4d5674ec83 100644
--- a/tools/crunchstat-summary/crunchstat_summary/summarizer.py
+++ b/tools/crunchstat-summary/crunchstat_summary/summarizer.py
@@ -422,7 +422,7 @@ class Summarizer(object):
         if asked_cores is None:
             asked_cores = 1
         # TODO: This should be more nuanced in cases where max >> avg
-        if used_cores < asked_cores:
+        if used_cores < (asked_cores*.5):
             yield recommendformat(
                 '{} max CPU usage was {}% -- '
                 'try reducing runtime_constraints to "{}":{}'
diff --git a/tools/crunchstat-summary/crunchstat_summary/webchart.py b/tools/crunchstat-summary/crunchstat_summary/webchart.py
index 9aedaa0445..f959661246 100644
--- a/tools/crunchstat-summary/crunchstat_summary/webchart.py
+++ b/tools/crunchstat-summary/crunchstat_summary/webchart.py
@@ -52,6 +52,9 @@ class WebChart(object):
           padding-right: 2em;
           border-top: 1px solid rgba(224, 224, 224, 1);
         }
+        #chart {
+          margin-left: -20px;
+        }
     '''
 
     def __init__(self, label, summarizers):

-----------------------------------------------------------------------


hooks/post-receive
--