Skip to content

Conversation

@BK1031
Copy link
Contributor

@BK1031 BK1031 commented Jan 8, 2026

Align TCMResourceTest1 expected values with the current 0x02A message decoder output to resolve test failures.

The previous test expectations assumed big-endian decoding for multi-byte fields and the presence of per-core CPU frequency/utilization signals. This PR updates the test to reflect the current little-endian decoding of multi-byte fields and the absence of per-core CPU signals in the 0x02A message output, allowing the test to pass.


Open in Cursor Open in Web


Note

Aligns TCMResourceTest1 with the current 0x02A message decoder to resolve test failures.

  • Removes per-core CPU signals (cpu_*_freq/cpu_*_util), retaining only cpu_total_util
  • Updates expected values and units for ram_total, ram_used, gpu_freq, disk_total, disk_used, voltage_draw, current_draw, and power_draw to match little-endian decoding and scaling

Written by Cursor Bugbot for commit 2d25707. This will update automatically on new commits. Configure here.

Co-authored-by: bharat1031 <bharat1031@gmail.com>
@cursor
Copy link

cursor bot commented Jan 8, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@netlify
Copy link

netlify bot commented Jan 8, 2026

Deploy Preview for gr-mpbench canceled.

Name Link
🔨 Latest commit 2d25707
🔍 Latest deploy log https://app.netlify.com/projects/gr-mpbench/deploys/69602943f4d08d0008e3e6b5

@BK1031
Copy link
Contributor Author

BK1031 commented Jan 8, 2026

alright lets see how good ai is with binary shit

@BK1031 BK1031 marked this pull request as ready for review January 8, 2026 22:10
@BK1031 BK1031 merged commit d68d0bd into main Jan 8, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants