Some proposed tweaks for process gpu and cpu stats

src/Managers/ProcessDRM.vala

@@ -30,7 +30,7 @@ public class Monitor.ProcessDRM : GLib.Object {
     private uint64 delta_ccs = 0;
     private uint64 delta_total_ccs = 0;
 
-    public double gpu_percentage { get; private set; }
+    public double gpu_percentage { get; private set; default = 0; }
 
     private int pid;
     private int update_interval;
@@ -99,7 +99,6 @@ public class Monitor.ProcessDRM : GLib.Object {
 
     public void update () {
         if (drm_files.size == 0) {
-            gpu_percentage = 0;
             return;
         }
 
@@ -138,7 +137,7 @@ public class Monitor.ProcessDRM : GLib.Object {
     }
 
     private void calculate_percentage_ns (ref uint64 engine, ref uint64 last_engine) {
-        if (last_engine != 0) {
+        if (last_engine != 0 && engine >= last_engine) {
             // Since values in the files are in nanoseconds, it is also needed to convert
             // the interval to nanoseconds (10^9)
             gpu_percentage = 100 * ((double) (engine - last_engine)) / (update_interval * 1e9);

src/Models/ProcessRowData.vala

@@ -9,9 +9,9 @@
 public class Monitor.ProcessRowData : GLib.Object {
     public Icon icon { get; set; }
     public string name { get; set; }
-    public int cpu { get; set; }
+    public double cpu { get; set; }
     public uint64 memory { get; set; }
-    public int gpu { get; set; }
+    public double gpu { get; set; }
     public int pid { get; set; }
     public string cmd { get; set; }
     public Gee.HashMap<string, Binding> bindings = new Gee.HashMap<string, Binding> ();

src/Models/TreeViewModel.vala

@@ -81,9 +81,9 @@ public class Monitor.TreeViewModel : GLib.Object {
             var row = new ProcessRowData () {
                 icon = process.icon,
                 name = process.application_name,
-                cpu = (int) process.cpu_percentage,
+                cpu = process.cpu_percentage,
                 memory = process.mem_usage,
-                gpu = (int) process.gpu_percentage,
+                gpu = process.gpu_percentage,
                 pid = process.stat.pid,
                 cmd = process.command
             };
@@ -111,9 +111,9 @@ public class Monitor.TreeViewModel : GLib.Object {
             }
 
             var item = (ProcessRowData) store.get_item (pos);
-            item.cpu = (int) process.cpu_percentage;
+            item.cpu = process.cpu_percentage;
             item.memory = process.mem_usage;
-            item.gpu = (int) process.gpu_percentage;
+            item.gpu = process.gpu_percentage;
             sorter.changed (DIFFERENT);
         }
     }

src/Views/ProcessView/ProcessTreeView/ProcessTreeView.vala

@@ -126,8 +126,8 @@ public class Monitor.ProcessTreeView : Granite.Bin {
         var label = (Gtk.Label) cell.child;
         var item = (ProcessRowData) cell.item;
         var binding_cpu = item.bind_property ("cpu", label, "label", SYNC_CREATE, (_, from_val, ref to_val) => {
-            int percentage = from_val.get_int ();
-            to_val.set_string ("%.0f%%".printf (percentage));
+            double percentage = from_val.get_double ();
+            to_val.set_string ("%.2f%%".printf (percentage));
             return true;
         });
         item.bindings.set ("cpu", binding_cpu);
@@ -165,8 +165,8 @@ public class Monitor.ProcessTreeView : Granite.Bin {
         var label = (Gtk.Label) cell.child;
         var item = (ProcessRowData) cell.item;
         var binding_gpu = item.bind_property ("gpu", label, "label", SYNC_CREATE, (_, from_val, ref to_val) => {
-            int percentage = from_val.get_int ();
-            to_val.set_string ("%.0f%%".printf (percentage));
+            double percentage = from_val.get_double ();
+            to_val.set_string ("%.2f%%".printf (percentage));
             return true;
         });
         item.bindings.set ("gpu", binding_gpu);

tests/test_process_drm.vala

@@ -30,7 +30,7 @@ private void test_process_drm () {
 
         drm.update ();
 
-        assert (drm.gpu_percentage == -1.0);
+        assert (drm.gpu_percentage < 0);
     });
 }