Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MIGraphX EP Support] Update migx scripts #21806

Merged
merged 2 commits into from
Aug 21, 2024

Conversation

TedThemistokleous
Copy link
Contributor

Description

No code changes to the EP only changes to the scripts whihc invoke MIGraphX EP

  • One case be explicit to set MIGraphX EP when running gpt2 testing
  • The other to ensure we turn off optimizations like tensorRT and allow MIGraphX to handle graph optimizations

Motivation and Context

MIGraphX has moved away from using rocBLAS and without this, some cases used in CI shall fail as optmizations will attempt to use rocBLAS kernels instead of MIGraphx EP directly.

@TedThemistokleous TedThemistokleous changed the title Update migx scripts [MIGraphX EP Support] Update migx scripts Aug 20, 2024
@TedThemistokleous
Copy link
Contributor Author

ping @cloudhan @tianleiwu @PeixuanZuo this doesnt effect the EP but testing. Useful to upstream this as it allows us to use your existing scripts correctly.

@tianleiwu
Copy link
Contributor

/azp run Big Models,Linux Android Emulator QNN CI Pipeline,Linux QNN CI Pipeline,ONNX Runtime Web CI Pipeline,Windows ARM64 QNN CI Pipeline,Windows GPU CUDA CI Pipeline,Windows GPU DML CI Pipeline,Windows GPU Doc Gen CI Pipeline,Windows x64 QNN CI Pipeline,onnxruntime-binary-size-checks-ci-pipeline

@tianleiwu
Copy link
Contributor

/azp run Windows GPU CI Pipeline, Linux CPU CI Pipeline, Linux CPU Minimal Build E2E CI Pipeline, Linux CPU x64 NoContribops CI Pipeline, Linux GPU CI Pipeline, Linux GPU TensorRT CI Pipeline, Linux OpenVINO CI Pipeline, MacOS CI Pipeline, MacOS NoContribops CI Pipeline, Windows CPU CI Pipeline

Copy link

Azure Pipelines successfully started running 10 pipeline(s).

Copy link

Azure Pipelines successfully started running 7 pipeline(s).

@tianleiwu
Copy link
Contributor

/azp run Windows GPU TensorRT CI Pipeline;orttraining-linux-ci-pipeline;orttraining-linux-gpu-ci-pipeline;orttraining-ortmodule-distributed

Copy link

No pipelines are associated with this pull request.

@cloudhan
Copy link
Contributor

/azp run Windows GPU TensorRT CI Pipeline,orttraining-linux-ci-pipeline,orttraining-linux-gpu-ci-pipeline,orttraining-ortmodule-distributed

Copy link

Azure Pipelines successfully started running 4 pipeline(s).

@tianleiwu tianleiwu merged commit 26a4993 into microsoft:main Aug 21, 2024
71 of 72 checks passed
@TedThemistokleous TedThemistokleous deleted the update_migx_scripts branch August 21, 2024 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants