Skip to content

Add GetVendorId for MiGraphX factory - #25515

Open
wonchung-microsoft wants to merge 1 commit into
mainfrom
dev/wonchung/migraphx_vendorid
Open

wonchung-microsoft wants to merge 1 commit into
mainfrom
dev/wonchung/migraphx_vendorid

Conversation

@wonchung-microsoft

Copy link
Copy Markdown
Contributor

No description provided.

: ort_api{ort_api_in}, ep_name{ep_name}, ort_hw_device_type{hw_type}, default_logger{default_logger_in} {
GetName = GetNameImpl;
GetVendor = GetVendorImpl;
GetVendorId = GetVendorIdImpl;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should also include the stubs that Scott is adding in #25503

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pradeep Sakhamoori (@psakhamoori) Would you please drive this comment and the pr merge?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wonchung-microsoft Please help add description to this PR

@psakhamoori

Copy link
Copy Markdown
Contributor

Nick Eubanks (@nieubank) to simplify and address implementation for MiGraphX w.r.t #PR 25503.

Created a single master PR # 25567

This PR # 25515 can be abandoned once # 25567 gets reviewed and merged

@jywu-mysoft George Wu (jywu-mysoft) added the ep:MIGraphX issues related to AMD MI GraphX execution provider label Jul 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ep:MIGraphX issues related to AMD MI GraphX execution provider

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants