LLNL / variorum

Vendor-neutral library for exposing power and performance features across diverse architectures

https://variorum.readthedocs.io

MIT License

68 stars 26 forks source link

Fix off-by-1 error in GPU utilization parsing in var_monitor #545

Closed tpatki closed 5 months ago

tpatki commented 5 months ago

Description

Fixes #544

Type of change

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature/architecture support (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Build/CI update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please provide hardware architecture specs and instructions so we can reproduce.

[X] Lassen GPU build

Checklist:

[X] I have run ./scripts/check-code-format.sh and confirm my code code follows the style guidelines of variorum
[X] I have added comments in my code
[X] My changes generate no new warnings (build with -DENABLE_WARNINGS=ON)
[X] New and existing unit tests pass with my changes

Thank you for taking the time to contribute to Variorum!