AI/rocm-systems
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix incorrect logging in mi_gpu_spec.py (#626)

Browse Source 
* Move console logging to logger function to avoid circular dependency in utils module

Signed-off-by: coleramos425 <colramos@amd.com>

* Apply python formatting

Signed-off-by: coleramos425 <colramos@amd.com>

* Remove the default StreamHandler before adding the custom

 If you are not explicitly removing this default handler, it could be causing duplicate outputs.

Signed-off-by: coleramos425 <colramos@amd.com>

* Fix lingering bugs from merge conflict resolution

Signed-off-by: coleramos425 <colramos@amd.com>

* Comply to python formatting and update pre-commit hook helper

Signed-off-by: coleramos425 <colramos@amd.com>

* Removing redundant console_log call as the get_mi300_num_xcds() call, otherwise ALL Mi200 profiling runs will print this message

Signed-off-by: coleramos425 <colramos@amd.com>

---------

Signed-off-by: coleramos425 <colramos@amd.com>

[ROCm/rocprofiler-compute commit: 04f92b72a9]
This commit is contained in:
Cole Ramos
2025-03-25 17:06:37 -05:00
committed by GitHub
parent a851c977c7
commit 088fa6a4ac
31 changed files with 132 additions and 129 deletions
Download Patch File Download Diff File Expand all files Collapse all files
projects/rocprofiler-compute/src/rocprof-compute
+1 -1
View File
@@ -36,7 +36,7 @@ try:
from importlib import metadata
from rocprof_compute_base import RocProfCompute
from utils.utils import console_error
from utils.logger import console_error
except ImportError as e:
# print("Failed to import required modules: " + str(e))
pass