Closed guoshzhao closed 8 months ago
Attention: 47 lines
in your changes are missing coverage. Please review.
Comparison is base (
6e50f02
) 86.12% compared to head (754dcea
) 85.78%.
Files | Patch % | Lines |
---|---|---|
superbench/common/utils/device_manager.py | 2.08% | 47 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
hi @guoshzhao, pls check these error msg from MI300
hi @guoshzhao, pls check these error msg from MI300
Thanks, just checked that GPU utilization and temperature APIs can work on MI250. Looks not supported on MI300. For GPU memory API, I have fixed it. For ECC API, the errors are expected, I have changed the log level to 'info'. Besides, I have change all other log level from 'error' to 'warning' to avoid the misunderstanding when incompitibility happens.
can we change the warning to only output once for each benchmark, there's too many warnings in the log by this
Description Upgrade to amdsmi python library since pyrsmi will be retired as AMD guys suggested:
AMD SMI Python Library: https://github.com/ROCm/amdsmi/tree/develop/py-interface pyrsmi: https://github.com/RadeonOpenCompute/pyrsmi