Skip to content

Commit

Permalink
[MIGraphX EP Support]Remove default noopt for Migraphx EP in Benchmar…
Browse files Browse the repository at this point in the history
…k.py (#21843)

…ripts (#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]>
  • Loading branch information
TedThemistokleous and Ted Themistokleous authored Aug 25, 2024
1 parent 87165b9 commit 9a70475
Showing 1 changed file with 0 additions and 9 deletions.
9 changes: 0 additions & 9 deletions onnxruntime/python/tools/transformers/benchmark.py
Original file line number Diff line number Diff line change
Expand Up @@ -135,15 +135,6 @@ def run_onnxruntime(
)
return results

if provider == "migraphx":
optimizer_info = OptimizerInfo.NOOPT
warm_up_repeat = 5
if "MIGraphXExecutionProvider" not in onnxruntime.get_available_providers():
logger.error(
"Please install onnxruntime-rocm package, and use a machine with GPU for testing gpu performance."
)
return results

if optimizer_info == OptimizerInfo.NOOPT:
logger.warning(
f"OptimizerInfo is set to {optimizer_info}, graph optimizations specified in FusionOptions are not applied."
Expand Down

0 comments on commit 9a70475

Please sign in to comment.