Minor fix

Author: terasakisatoshi

ext/TerminalSystemMonitorCUDAExt/TerminalSystemMonitorCUDAExt.jl

@@ -56,8 +56,11 @@ end
 
 function TerminalSystemMonitor.plot_gpu_utilization_rates(::Type{CUDADevice})
     plts = []
-    for (jli, dev) in enumerate(CUDA.devices())
-        gpu_id = jli - 1 # from 1-based to 0-based index
+    devices = collect(CUDA.devices())
+    npad = 1 + floor(Int, log10(length(devices)))
+    xs = ["id: $(lpad(i-1, npad))" for i = 1:length(devices)]
+
+    for (gpu_id, dev) in zip(xs, devices)
         push!(plts, _plot_gpu_utilization_rates(gpu_id, dev))
     end
     return plts

src/TerminalSystemMonitor.jl

@@ -73,20 +73,20 @@ function extract_number_and_unit(str::AbstractString)
 end
 
 function plot_cpu_utilization_rates()
-    y = get_cpu_percent()
-    npad = 1 + floor(Int, log10(length(y)))
-    x = ["id: $(lpad(i-1, npad))" for (i, _) in enumerate(y)]
+    ys = get_cpu_percent()
+    npad = 1 + floor(Int, log10(length(ys)))
+    xs = ["id: $(lpad(i-1, npad))" for (i, _) in enumerate(ys)]
 
-    ncpus = length(y)
-    y = round.(y, digits = 1)
+    ncpus = length(ys)
+    ys = round.(ys, digits = 1)
 
     plts = []
 
     chunks = collect.(collect(Iterators.partition((1:ncpus), 4)))
     for c in chunks
         push!(
             plts,
-            barplot(x[c], y[c], maximum = 100, width = max(5, 15), height = length(c)),
+            barplot(xs[c], ys[c], maximum = 100, width = 15, height = length(c)),
         )
     end
     return plts
@@ -120,7 +120,7 @@ function plot_cpu_memory_utilization()
             # Adds a space for better styling
             name = " $(memorytotal) $(memorytotal_unit)",
             maximum = memorytotal,
-            width = max(5, 15),
+            width = 30,
         ),
     )
 
@@ -140,7 +140,8 @@ function main(dummyargs...)
 
             f /= prod(UnicodePlots.panel.(plot_cpu_memory_utilization()))
 
-            if isdefined(Main, :CUDA) && getproperty(getproperty(Main, :CUDA), :functional)()
+            if isdefined(Main, :CUDA) &&
+               getproperty(getproperty(Main, :CUDA), :functional)()
                 cudaplts = []
                 n = max(1, cols ÷ 50)
                 plts1::Vector{Any} = plot_gpu_utilization_rates(MLDataDevices.CUDADevice)