Skip to content

Commit 2bdca62

Browse files
follesoeclaude
andcommitted
Address Copilot review: capitalize CPU, rename cpu_load_average
- Capitalize "cpu" to "CPU" in telemetry deprecation comment for consistency with existing naming conventions. - Rename cpu_load_average to cpu_utilization to avoid confusion with Linux load average (which is unbounded and reported as 1/5/15 min). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent c41109f commit 2bdca62

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

protobuf_definitions/message_formats.proto

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1368,7 +1368,7 @@ message PowerRailInfo {
13681368
// Fields not applicable to a platform are left at their zero/empty defaults.
13691369
message SystemPerformanceInfo {
13701370
repeated CpuCoreLoad cpu_cores = 1; // Per-core CPU utilization.
1371-
float cpu_load_average = 2; // Overall CPU load (0..1).
1371+
float cpu_utilization = 2; // Mean CPU utilization across all cores (0..1).
13721372
GpuInfo gpu = 3; // GPU utilization.
13731373
repeated DlaInfo dla_engines = 4; // DLA engine utilization (Jetson only).
13741374
MemoryInfo memory = 5; // RAM and swap usage.

protobuf_definitions/telemetry.proto

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -295,7 +295,7 @@ message MultibeamDiscoveryTel {
295295
MultibeamDiscovery discovery = 1; // Discovery data from a multibeam sonar.
296296
}
297297

298-
// Information about cpu and memory usage (deprecated, use SystemPerformanceInfoTel instead).
298+
// Information about CPU and memory usage (deprecated, use SystemPerformanceInfoTel instead).
299299
message CPUInfoTel {
300300
CPUInfo cpu_info = 1; // CPU information.
301301
}

0 commit comments

Comments
 (0)