From cb482fa3eacaebe68eff11112ac235a764d898a8 Mon Sep 17 00:00:00 2001
From: Rodrigo Arias Mallo <rodrigo.arias@bsc.es>
Date: Wed, 17 Mar 2021 20:12:27 +0100
Subject: [PATCH 1/9] heat: remove perf from the ctf experiment

As we would be extracting perf stats from the trace processing steps.
---
 garlic/exp/index.nix | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/garlic/exp/index.nix b/garlic/exp/index.nix
index 5645c625..c930385e 100644
--- a/garlic/exp/index.nix
+++ b/garlic/exp/index.nix
@@ -66,7 +66,7 @@
   heat = rec {
     granul = callPackage ./heat/granul.nix { };
     cache = granul.override { enablePerf = true; };
-    ctf = cache.override { enableCTF = true; };
+    ctf = granul.override { enableCTF = true; };
   };
 
   bigsort = rec {
-- 
GitLab


From d68ce914babdf0a00b24f3e1a0b23f90defb4523 Mon Sep 17 00:00:00 2001
From: Rodrigo Arias Mallo <rodrigo.arias@bsc.es>
Date: Wed, 17 Mar 2021 20:13:49 +0100
Subject: [PATCH 2/9] heat: use cut to partition the trace

The awk script doesn't take in consideration the events close to the cut
points, which are significative with low parallelism.
---
 garlic/exp/heat/granul.nix | 24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/garlic/exp/heat/granul.nix b/garlic/exp/heat/granul.nix
index 6fab7b62..70bcd317 100644
--- a/garlic/exp/heat/granul.nix
+++ b/garlic/exp/heat/granul.nix
@@ -15,12 +15,12 @@ with garlicTools;
 let
   # Initial variable configuration
   varConf = with bsc; {
-    #cbs = range2 32 4096;
-    #rbs = range2 32 4096;
-    cbs = [ 64 256 1024 4096 ];
-    rbs = [ 32 128 512 1024 ];
+    cbs = range2 32 4096;
+    rbs = range2 32 4096;
+    #cbs = [ 64 256 1024 4096 ];
+    #rbs = [ 32 128 512 1024 ];
     #cbs = [ 4096 ];
-    #rbs = [ 32 ];
+    #rbs = [ 512 ];
   };
 
   machineConfig = targetMachine.config;
@@ -94,23 +94,23 @@ let
 
           ${bsc.cn6}/bin/cn6 -s $tracedir
 
-          awk -F: "NR==1 {print} \$6 >= $begin && \$6 <= $end" $tracedir/prv/trace.prv |\
-            ${bsc.cn6}/bin/dur 6400025 0 |\
+          ${bsc.cn6}/bin/cut $begin $end \
+            < $tracedir/prv/trace.prv \
+            > $tracedir/prv/trace-cut.prv
+
+          ${bsc.cn6}/bin/dur 6400025 0 < $tracedir/prv/trace-cut.prv |\
             awk '{s+=$1} END {print s}' >> .garlic/time_mode_dead.csv &
 
-          awk -F: "NR==1 {print} \$6 >= $begin && \$6 <= $end" $tracedir/prv/trace.prv |\
-            ${bsc.cn6}/bin/dur 6400025 1 |\
+          ${bsc.cn6}/bin/dur 6400025 1 < $tracedir/prv/trace-cut.prv |\
             awk '{s+=$1} END {print s}' >> .garlic/time_mode_runtime.csv &
 
-          awk -F: "NR==1 {print} \$6 >= $begin && \$6 <= $end" $tracedir/prv/trace.prv |\
-            ${bsc.cn6}/bin/dur 6400025 3 |\
+          ${bsc.cn6}/bin/dur 6400025 3 < $tracedir/prv/trace-cut.prv |\
             awk '{s+=$1} END {print s}' >> .garlic/time_mode_task.csv &
 
           wait
 
           # Remove the traces at the end, as they are huge
           rm -rf $tracedir
-          #cp -a $tracedir .garlic/
         done
       fi
     '';
-- 
GitLab


From 699404bafe26902aa4115225d42c47902e5e2da9 Mon Sep 17 00:00:00 2001
From: Rodrigo Arias Mallo <rodrigo.arias@bsc.es>
Date: Thu, 18 Mar 2021 20:03:44 +0100
Subject: [PATCH 3/9] bsc: add cpuid program

---
 bsc/cpuid/default.nix | 21 +++++++++++++++++++++
 overlay.nix           |  1 +
 2 files changed, 22 insertions(+)
 create mode 100644 bsc/cpuid/default.nix

diff --git a/bsc/cpuid/default.nix b/bsc/cpuid/default.nix
new file mode 100644
index 00000000..30b9edc6
--- /dev/null
+++ b/bsc/cpuid/default.nix
@@ -0,0 +1,21 @@
+{
+  stdenv
+, perl # For the pod2man command
+}:
+
+stdenv.mkDerivation rec {
+  version = "20201006";
+  pname = "cpuid";
+
+  buildInputs = [ perl ];
+
+  # Replace /usr install directory for $out
+  postPatch = ''
+    sed -i "s@/usr@$out@g" Makefile
+  '';
+
+  src = builtins.fetchTarball {
+    url = "http://www.etallen.com/cpuid/${pname}-${version}.src.tar.gz";
+    sha256 = "04qhs938gs1kjxpsrnfy6lbsircsprfyh4db62s5cf83a1nrwn9w";
+  };
+}
diff --git a/overlay.nix b/overlay.nix
index 8e4b01ab..75c18818 100644
--- a/overlay.nix
+++ b/overlay.nix
@@ -197,6 +197,7 @@ let
 
     dummy = callPackage ./bsc/dummy/default.nix { };
     mpptest = callPackage ./bsc/mpptest/default.nix { };
+    cpuid = callPackage ./bsc/cpuid/default.nix { };
 
     # =================================================================
     #  Garlic benchmark
-- 
GitLab


From f8122f3c8b1c8ca86b446ee9a721e7c5a56251f7 Mon Sep 17 00:00:00 2001
From: Rodrigo Arias Mallo <rodrigo.arias@bsc.es>
Date: Thu, 18 Mar 2021 20:06:53 +0100
Subject: [PATCH 4/9] heat: use the hcut tool to limit the cpus

---
 garlic/exp/heat/granul.nix | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/garlic/exp/heat/granul.nix b/garlic/exp/heat/granul.nix
index 70bcd317..dbf31470 100644
--- a/garlic/exp/heat/granul.nix
+++ b/garlic/exp/heat/granul.nix
@@ -94,8 +94,8 @@ let
 
           ${bsc.cn6}/bin/cn6 -s $tracedir
 
-          ${bsc.cn6}/bin/cut $begin $end \
-            < $tracedir/prv/trace.prv \
+          ${bsc.cn6}/bin/cut $begin $end < $tracedir/prv/trace.prv |\
+            ${bsc.cn6}/bin/hcut 1 ${toString conf.cpusPerTask} \
             > $tracedir/prv/trace-cut.prv
 
           ${bsc.cn6}/bin/dur 6400025 0 < $tracedir/prv/trace-cut.prv |\
-- 
GitLab


From 0b7e92b6f99271a44b6b39876815db1f4558579e Mon Sep 17 00:00:00 2001
From: Rodrigo Arias Mallo <rodrigo.arias@bsc.es>
Date: Thu, 18 Mar 2021 20:08:24 +0100
Subject: [PATCH 5/9] heat: add bar plot with time distribution

---
 garlic/fig/heat/mode.R | 48 ++++++++++++++++++++++++++++++++++++++----
 1 file changed, 44 insertions(+), 4 deletions(-)

diff --git a/garlic/fig/heat/mode.R b/garlic/fig/heat/mode.R
index 66bde822..8e7e3ac1 100644
--- a/garlic/fig/heat/mode.R
+++ b/garlic/fig/heat/mode.R
@@ -3,6 +3,7 @@ library(dplyr)
 library(scales)
 library(jsonlite)
 library(viridis)
+library(tidyr)
 
 args=commandArgs(trailingOnly=TRUE)
 
@@ -19,6 +20,7 @@ df = select(dataset, config.cbs, config.rbs,
     ctf_mode.runtime,
     ctf_mode.task,
     ctf_mode.dead,
+    config.cpusPerTask,
     time) %>%
 	rename(
     cbs=config.cbs,
@@ -26,6 +28,7 @@ df = select(dataset, config.cbs, config.rbs,
     runtime=ctf_mode.runtime,
     task=ctf_mode.task,
     dead=ctf_mode.dead,
+    cpusPerTask=config.cpusPerTask,
   )
 
 df$cbs = as.factor(df$cbs)
@@ -33,16 +36,16 @@ df$rbs = as.factor(df$rbs)
 
 # Normalize the time by the median
 df = df %>%
-	mutate(runtime = runtime * 1e-9) %>%
-	mutate(dead = dead * 1e-9) %>%
-	mutate(task = task * 1e-9) %>%
+	mutate(runtime = runtime * 1e-9 / cpusPerTask) %>%
+	mutate(dead = dead * 1e-9 / cpusPerTask) %>%
+	mutate(task = task * 1e-9 / cpusPerTask) %>%
   group_by(cbs, rbs) %>%
 	mutate(median.time = median(time)) %>%
 	mutate(log.median.time = log(median.time)) %>%
 	mutate(median.dead = median(dead)) %>%
 	mutate(median.runtime = median(runtime)) %>%
 	mutate(median.task = median(task)) %>%
-  ungroup()# %>%
+  ungroup() #%>%
 
 print(df)
 
@@ -79,3 +82,40 @@ df_filtered = filter(df, between(median.time,
 
 heatmap_plot(df, "median.time", "execution time (seconds)")
 heatmap_plot(df, "log.median.time", "execution time")
+
+df_square = filter(df, cbs == rbs) %>%
+  gather(key = time.from, value = acc.time, 
+    c("median.dead", "median.runtime", "median.task"))
+
+# Colors similar to Paraver
+colors <- c("median.dead" = "gray",
+  "median.runtime" = "blue",
+  "median.task" = "red")
+
+p = ggplot(df_square, aes(x=cbs, y=acc.time)) +
+  geom_area(aes(fill=time.from, group=time.from)) +
+  scale_fill_manual(values = colors) +
+  geom_point(aes(y=median.time, color="black")) +
+  geom_line(aes(y=median.time, group=0, color="black")) +
+  theme_bw() +
+  theme(legend.position=c(0.5, 0.7)) +
+  scale_color_identity(breaks = c("black"),
+    labels = c("Total time"), guide = "legend") +
+  labs(x="Blocksize (side)", y="Time (s)",
+    fill="Estimated", color="Direct measurement",
+    title="Heat granularity: time distribution", subtitle=input_file)
+
+ggsave("area.time.png", plot=p, width=6, height=6, dpi=300)
+ggsave("area.time.pdf", plot=p, width=6, height=6, dpi=300)
+
+p = ggplot(df_square, aes(x=cbs, y=acc.time)) +
+  geom_col(aes(fill=time.from, group=time.from)) +
+  scale_fill_manual(values = colors) +
+  theme_bw() +
+  theme(legend.position=c(0.5, 0.7)) +
+  labs(x="Blocksize (side)", y="Time (s)",
+    fill="Estimated", color="Direct measurement",
+    title="Heat granularity: time distribution", subtitle=input_file)
+
+ggsave("col.time.png", plot=p, width=6, height=6, dpi=300)
+ggsave("col.time.pdf", plot=p, width=6, height=6, dpi=300)
-- 
GitLab


From 3566cf0152c256a8a2152cc531691c5de9ff1246 Mon Sep 17 00:00:00 2001
From: Rodrigo Arias Mallo <rodrigo.arias@bsc.es>
Date: Tue, 6 Apr 2021 11:14:30 +0200
Subject: [PATCH 6/9] develop: add paraver package

---
 garlic/index.nix | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/garlic/index.nix b/garlic/index.nix
index e55fb386..b73c86d4 100644
--- a/garlic/index.nix
+++ b/garlic/index.nix
@@ -14,7 +14,7 @@
       # Add more nixpkgs packages here...
     ];
     bscPackages = with bsc; [
-      slurm clangOmpss2 icc mcxx perf tampi impi vtk
+      slurm clangOmpss2 icc mcxx perf tampi impi vtk paraver
       # Add more bsc packages here...
     ];
     packages = commonPackages ++ bscPackages;
-- 
GitLab


From d1c32869c155f388dff844ed5ef4a96e7f2bc1bc Mon Sep 17 00:00:00 2001
From: Rodrigo Arias Mallo <rodrigo.arias@bsc.es>
Date: Tue, 6 Apr 2021 18:38:15 +0200
Subject: [PATCH 7/9] heat: split granularity with extended mode

The HWC version is not yet complete.
---
 garlic/exp/heat/granul.nix | 116 +++++++++++++++++++++++--------------
 1 file changed, 72 insertions(+), 44 deletions(-)

diff --git a/garlic/exp/heat/granul.nix b/garlic/exp/heat/granul.nix
index dbf31470..5d4d01ee 100644
--- a/garlic/exp/heat/granul.nix
+++ b/garlic/exp/heat/granul.nix
@@ -5,10 +5,16 @@
 , targetMachine
 , stages
 , garlicTools
+, writeText
 , enablePerf ? false
 , enableCTF ? false
+, enableHWC ? false
+, enableExtended ? false
 }:
 
+# TODO: Finish HWC first
+assert (enableHWC == false);
+
 with stdenv.lib;
 with garlicTools;
 
@@ -17,10 +23,6 @@ let
   varConf = with bsc; {
     cbs = range2 32 4096;
     rbs = range2 32 4096;
-    #cbs = [ 64 256 1024 4096 ];
-    #rbs = [ 32 128 512 1024 ];
-    #cbs = [ 4096 ];
-    #rbs = [ 512 ];
   };
 
   machineConfig = targetMachine.config;
@@ -38,12 +40,11 @@ let
     timesteps = 10;
     cols = 1024 * 16; # Columns
     rows = 1024 * 16; # Rows
-    cbs = c.cbs;
-    rbs = c.rbs;
+    inherit (c) cbs rbs;
     gitBranch = "garlic/tampi+isend+oss+task";
     
     # Repeat the execution of each unit 30 times
-    loops = 1;
+    loops = 10;
 
     # Resources
     qos = "debug";
@@ -55,10 +56,20 @@ let
     jobName = unitName;
   };
 
+  filterConfigs = c: let
+    # Too small sizes lead to huge overheads
+    goodSize = (c.cbs * c.rbs >= 1024);
+    # When the extended units are not enabled, we only select those in
+    # the diagonal.
+    extended = if (enableExtended) then true
+      else c.cbs == c.rbs;
+  in
+    goodSize && extended;
+
   # Compute the array of configurations
-  configs = stdexp.buildConfigs {
+  configs = filter (filterConfigs) (stdexp.buildConfigs {
     inherit varConf genConf;
-  };
+  });
 
   perf = {nextStage, conf, ...}: stages.perf {
     inherit nextStage;
@@ -66,54 +77,71 @@ let
       "-e cycles,instructions,cache-references,cache-misses";
   };
 
-  ctf = {nextStage, conf, ...}: with conf; stages.exec {
+  ctf = {nextStage, conf, ...}: let
+    # Create the nanos6 configuration file
+    nanos6ConfigFile = writeText "nanos6.toml" ''
+      version.instrument = "ctf"
+      turbo.enabled = false
+      instrument.ctf.converter.enabled = false
+    '' + optionalString (enableHWC) ''
+      hardware_counters.papi.enabled = true
+      hardware_counters.papi.counters = [
+        "PAPI_TOT_INS", "PAPI_TOT_CYC",
+        "PAPI_L1_TCM", "PAPI_L2_TCM", "PAPI_L3_TCM"
+      ]
+    '';
+
+  in stages.exec {
     inherit nextStage;
+
+    # And use it
     env = ''
-      export NANOS6_CONFIG_OVERRIDE="version.instrument=ctf,\
-        instrument.ctf.converter.enabled=false"
+      export NANOS6_CONFIG=${nanos6ConfigFile}
     '';
-    # Only one process converts the trace, otherwise use:
-    #  if [ $SLURM_PROCID == 0 ]; then
-    #    ...
-    #  fi
-    post = ''
-      if [ $SLURM_PROCID == 0 ]; then
-        sleep 2
-        for tracedir in trace_*; do
-          offset=$(grep 'offset =' $tracedir/ctf/ust/uid/1000/64-bit/metadata | \
-            grep -o '[0-9]*')
-          echo "offset = $offset"
 
-          start_time=$(awk '/^start_time / {print $2}' stdout.log)
-          end_time=$(awk '/^end_time / {print $2}' stdout.log)
+    # FIXME: We should run a hook *after* srun has ended, so we can
+    # execute it in one process only (not in N ranks). This hack works
+    # with one process only. Or be able to compute the name of the trace
+    # directory so we can begin the conversion in parallel
+    post = assert (conf.nodes * conf.ntasksPerNode == 1); ''
+      tracedir=$(ls -d trace_* | head -1)
+      echo "using tracedir=$tracedir"
 
-          begin=$(awk "BEGIN{print $start_time*1e9 - $offset}")
-          end=$(awk "BEGIN{print $end_time*1e9 - $offset}")
+      offset=$(grep 'offset =' $tracedir/ctf/ust/uid/1000/64-bit/metadata | \
+        grep -o '[0-9]*')
+      echo "offset = $offset"
 
-          echo "only events between $begin and $end"
+      start_time=$(awk '/^start_time / {print $2}' stdout.log)
+      end_time=$(awk '/^end_time / {print $2}' stdout.log)
 
-          ${bsc.cn6}/bin/cn6 -s $tracedir
+      begin=$(awk "BEGIN{print $start_time*1e9 - $offset}")
+      end=$(awk "BEGIN{print $end_time*1e9 - $offset}")
 
-          ${bsc.cn6}/bin/cut $begin $end < $tracedir/prv/trace.prv |\
-            ${bsc.cn6}/bin/hcut 1 ${toString conf.cpusPerTask} \
-            > $tracedir/prv/trace-cut.prv
+      echo "only events between $begin and $end"
 
-          ${bsc.cn6}/bin/dur 6400025 0 < $tracedir/prv/trace-cut.prv |\
-            awk '{s+=$1} END {print s}' >> .garlic/time_mode_dead.csv &
+      ${bsc.cn6}/bin/cn6 -s $tracedir
 
-          ${bsc.cn6}/bin/dur 6400025 1 < $tracedir/prv/trace-cut.prv |\
-            awk '{s+=$1} END {print s}' >> .garlic/time_mode_runtime.csv &
+      ${bsc.cn6}/bin/cut $begin $end < $tracedir/prv/trace.prv |\
+        ${bsc.cn6}/bin/hcut 1 ${toString conf.cpusPerTask} \
+        > $tracedir/prv/trace-cut.prv
 
-          ${bsc.cn6}/bin/dur 6400025 3 < $tracedir/prv/trace-cut.prv |\
-            awk '{s+=$1} END {print s}' >> .garlic/time_mode_task.csv &
+      ${bsc.cn6}/bin/dur 6400025 0 < $tracedir/prv/trace-cut.prv |\
+        awk '{s+=$1} END {print s}' >> .garlic/time_mode_dead.csv &
 
-          wait
+      ${bsc.cn6}/bin/dur 6400025 1 < $tracedir/prv/trace-cut.prv |\
+        awk '{s+=$1} END {print s}' >> .garlic/time_mode_runtime.csv &
 
-          # Remove the traces at the end, as they are huge
-          rm -rf $tracedir
-        done
-      fi
-    '';
+      ${bsc.cn6}/bin/dur 6400025 3 < $tracedir/prv/trace-cut.prv |\
+        awk '{s+=$1} END {print s}' >> .garlic/time_mode_task.csv &
+
+      wait
+
+      # Remove the traces at the end, as they are huge
+      rm -rf $tracedir
+      '';
+      # TODO: To enable HWC we need to first add a taskwait before the
+      # first get_time() measurement, otherwise we get the HWC of the
+      # main task, which will be huge.
   };
 
   exec = {nextStage, conf, ...}: stages.exec {
-- 
GitLab


From 63aa07dad518235894014367e4fdd6115550d749 Mon Sep 17 00:00:00 2001
From: Rodrigo Arias Mallo <rodrigo.arias@bsc.es>
Date: Tue, 6 Apr 2021 18:40:19 +0200
Subject: [PATCH 8/9] heat: update granularity plot with modern ggplot

---
 garlic/fig/heat/granul.R | 151 +++++++++++++--------------------------
 1 file changed, 49 insertions(+), 102 deletions(-)

diff --git a/garlic/fig/heat/granul.R b/garlic/fig/heat/granul.R
index d3dc5d71..fb72e009 100644
--- a/garlic/fig/heat/granul.R
+++ b/garlic/fig/heat/granul.R
@@ -1,120 +1,67 @@
 library(ggplot2)
-library(dplyr)
+library(dplyr, warn.conflicts = FALSE)
 library(scales)
 library(jsonlite)
+library(viridis, warn.conflicts = FALSE)
+library(stringr)
 
-args=commandArgs(trailingOnly=TRUE)
+args = commandArgs(trailingOnly=TRUE)
 
-# Read the timetable from args[1]
-input_file = "input.json"
-if (length(args)>0) { input_file = args[1] }
+# Set the input dataset if given in argv[1], or use "input" as default
+if (length(args)>0) { input_file = args[1] } else { input_file = "input" }
 
-# Load the dataset in NDJSON format
-dataset = jsonlite::stream_in(file(input_file)) %>%
-	jsonlite::flatten()
+df = jsonlite::stream_in(file(input_file), verbose=FALSE) %>%
 
+  jsonlite::flatten() %>%
 
-# We only need the nblocks and time
-df = select(dataset, config.cbs, config.rbs, time) %>%
-	rename(cbs=config.cbs, rbs=config.rbs)
+  select(unit,
+    config.cbs,
+    config.rbs,
+    time,
+    total_time) %>%
 
-df$cbs = as.factor(df$cbs)
-df$rbs = as.factor(df$rbs)
+  rename(cbs=config.cbs,
+    rbs=config.rbs) %>%
 
-# Normalize the time by the median
-df=group_by(df, cbs, rbs) %>%
-	mutate(mtime = median(time)) %>%
-	mutate(tnorm = time / mtime - 1) %>%
-	mutate(logmtime = log(mtime)) %>%
-  ungroup() %>%
-  filter(between(mtime, mean(time) - (1 * sd(time)),
-    mean(time) + (1 * sd(time))))
+  # Convert to factors
+  mutate(cbs = as.factor(cbs)) %>%
+  mutate(rbs = as.factor(rbs)) %>%
+  mutate(unit = as.factor(unit)) %>%
 
-ppi=300
-h=5
-w=5
+  # Compute median times
+  group_by(unit) %>%
+  mutate(median.time = median(time)) %>%
+  mutate(normalized.time = time / median.time - 1) %>%
+  mutate(log.median.time = log(median.time)) %>%
+  ungroup()
 
-png("box.png", width=w*ppi, height=h*ppi, res=ppi)
-#
-#
-#
-# Create the plot with the normalized time vs nblocks
-p = ggplot(data=df, aes(x=cbs, y=tnorm)) +
+dpi = 300
+h = 6
+w = 6
 
-	# Labels
-	labs(x="cbs", y="Normalized time",
-              title=sprintf("Heat normalized time"), 
-              subtitle=input_file) +
+# ---------------------------------------------------------------------
 
-	# Center the title
-	#theme(plot.title = element_text(hjust = 0.5)) +
+p = ggplot(df, aes(x=cbs, y=normalized.time)) +
+  geom_boxplot() +
+  geom_hline(yintercept=c(-0.01, 0.01), linetype="dashed", color="red") +
+  theme_bw() +
+  labs(y="Normalized time",
+       title="Heat granularity: normalized time", 
+    subtitle=input_file) + 
+  theme(plot.subtitle=element_text(size=8))
 
-	# Black and white mode (useful for printing)
-	#theme_bw() +
+ggsave("normalized.time.png", plot=p, width=w, height=h, dpi=dpi)
+ggsave("normalized.time.pdf", plot=p, width=w, height=h, dpi=dpi)
 
-	# Add the maximum allowed error lines
-	geom_hline(yintercept=c(-0.01, 0.01),
-		linetype="dashed", color="red") +
+# ---------------------------------------------------------------------
 
-	# Draw boxplots
-	geom_boxplot() +
+p = ggplot(df, aes(x=cbs, y=time)) +
+  geom_point(shape=21, size=3) +
+  geom_line(aes(y=median.time, group=0)) +
+  theme_bw() +
+  labs(y="Time (s)", title="Heat granularity: time", 
+    subtitle=input_file) + 
+  theme(plot.subtitle=element_text(size=8))
 
-	#scale_y_continuous(breaks = scales::pretty_breaks(n = 10)) +
-
-	theme_bw() +
-
-	theme(plot.subtitle=element_text(size=8)) +
-
-	theme(legend.position = c(0.85, 0.85)) #+
-
-
-
-
-# Render the plot
-print(p)
-
-## Save the png image
-dev.off()
-#
-png("scatter.png", width=w*ppi, height=h*ppi, res=ppi)
-#
-## Create the plot with the normalized time vs nblocks
-p = ggplot(df, aes(x=cbs, y=time, linetype=rbs, group=rbs)) +
-
-	labs(x="cbs", y="Time (s)",
-              title=sprintf("Heat granularity"), 
-              subtitle=input_file) +
-	theme_bw() +
-	theme(plot.subtitle=element_text(size=8)) +
-	theme(legend.position = c(0.5, 0.88)) +
-
-	geom_point(shape=21, size=3) +
-	geom_line(aes(y=mtime)) +
-	#scale_x_continuous(trans=log2_trans()) +
-	scale_y_continuous(trans=log2_trans())
-
-# Render the plot
-print(p)
-
-# Save the png image
-dev.off()
-
-
-png("heatmap.png", width=w*ppi, height=h*ppi, res=ppi)
-#
-## Create the plot with the normalized time vs nblocks
-p = ggplot(df, aes(x=cbs, y=rbs, fill=logmtime)) +
-	geom_raster() +
-  scale_fill_gradient(high="black", low="white") +
-  coord_fixed() +
-	theme_bw() +
-	theme(plot.subtitle=element_text(size=8)) +
-	labs(x="cbs", y="rbs",
-    title=sprintf("Heat granularity"), 
-    subtitle=input_file)
-
-# Render the plot
-print(p)
-
-# Save the png image
-dev.off()
+ggsave("time.png", plot=p, width=w, height=h, dpi=dpi)
+ggsave("time.pdf", plot=p, width=w, height=h, dpi=dpi)
-- 
GitLab


From 312656ce5460b7ddc087f5c135879b21e80849a0 Mon Sep 17 00:00:00 2001
From: Rodrigo Arias Mallo <rodrigo.arias@bsc.es>
Date: Tue, 6 Apr 2021 18:42:49 +0200
Subject: [PATCH 9/9] heat: rename granul -> granularity experiment

---
 garlic/exp/heat/{granul.nix => granularity.nix} | 0
 garlic/exp/index.nix                            | 6 +++---
 garlic/fig/heat/{granul.R => granularity.R}     | 0
 garlic/fig/index.nix                            | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)
 rename garlic/exp/heat/{granul.nix => granularity.nix} (100%)
 rename garlic/fig/heat/{granul.R => granularity.R} (100%)

diff --git a/garlic/exp/heat/granul.nix b/garlic/exp/heat/granularity.nix
similarity index 100%
rename from garlic/exp/heat/granul.nix
rename to garlic/exp/heat/granularity.nix
diff --git a/garlic/exp/index.nix b/garlic/exp/index.nix
index c930385e..3a007fa4 100644
--- a/garlic/exp/index.nix
+++ b/garlic/exp/index.nix
@@ -64,9 +64,9 @@
   };
 
   heat = rec {
-    granul = callPackage ./heat/granul.nix { };
-    cache = granul.override { enablePerf = true; };
-    ctf = granul.override { enableCTF = true; };
+    granularity = callPackage ./heat/granularity.nix { };
+    cache = granularity.override { enablePerf = true; };
+    ctf = granularity.override { enableCTF = true; };
   };
 
   bigsort = rec {
diff --git a/garlic/fig/heat/granul.R b/garlic/fig/heat/granularity.R
similarity index 100%
rename from garlic/fig/heat/granul.R
rename to garlic/fig/heat/granularity.R
diff --git a/garlic/fig/index.nix b/garlic/fig/index.nix
index 81244d02..c36f0b44 100644
--- a/garlic/fig/index.nix
+++ b/garlic/fig/index.nix
@@ -47,7 +47,7 @@ in
   };
 
   heat = with exp.heat; {
-    granul = stdPlot ./heat/granul.R [ granul ];
+    granularity = stdPlot ./heat/granularity.R [ granularity ];
     cache = customPlot ./heat/cache.R (ds.perf.stat cache.result);
     ctf = customPlot ./heat/mode.R (ds.ctf.mode ctf.result);
   };
-- 
GitLab