Skip to content

Commit 9a70475

Browse files
TedThemistokleousTed Themistokleous
andauthored
[MIGraphX EP Support]Remove default noopt for Migraphx EP in Benchmark.py (microsoft#21843)
…ripts (microsoft#58) ### Description <!-- Describe your changes. --> Removes the heavy handed no opt for all MIGraphX using the benchmark.py scripts ### Motivation and Context <!-- - Why is this change required? What problem does it solve? - If it fixes an open issue, please link to the issue here. --> Finding this hurts performance if we remove all optimizations. Let the fine tuning occur at the script level instead of a blanket NoOPT being selected Co-authored-by: Ted Themistokleous <[email protected]>
1 parent 87165b9 commit 9a70475

File tree

1 file changed

+0
-9
lines changed

1 file changed

+0
-9
lines changed

onnxruntime/python/tools/transformers/benchmark.py

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -135,15 +135,6 @@ def run_onnxruntime(
135135
)
136136
return results
137137

138-
if provider == "migraphx":
139-
optimizer_info = OptimizerInfo.NOOPT
140-
warm_up_repeat = 5
141-
if "MIGraphXExecutionProvider" not in onnxruntime.get_available_providers():
142-
logger.error(
143-
"Please install onnxruntime-rocm package, and use a machine with GPU for testing gpu performance."
144-
)
145-
return results
146-
147138
if optimizer_info == OptimizerInfo.NOOPT:
148139
logger.warning(
149140
f"OptimizerInfo is set to {optimizer_info}, graph optimizations specified in FusionOptions are not applied."

0 commit comments

Comments
 (0)