Skip to content

Conversation

@adrastogi
Copy link
Contributor

Description

This change corrects the signatures on the new precompiled model compatibility APIs to use the recommended macros.

Motivation and Context

This fix addresses PR feedback from #25454 .

jywu-msft
jywu-msft previously approved these changes Jul 22, 2025
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

You can commit the suggested changes from lintrunner.

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@skottmckay skottmckay merged commit 8446e04 into main Jul 23, 2025
91 of 95 checks passed
@skottmckay skottmckay deleted the adrastogi/compat-api-annotations branch July 23, 2025 03:08
@snnn
Copy link
Member

snnn commented Jul 25, 2025

Hi there! We haven't cut the release branch for this version yet, so I'm removing the release:1.23.0 label for now to keep things tidy. Thanks so much for your contribution! We'll make sure this gets included when the release is prepared. 🤖

sanketkaleoss pushed a commit to sanketkaleoss/onnxruntime that referenced this pull request Aug 11, 2025
…osoft#25497)

### Description
This change corrects the signatures on the new precompiled model
compatibility APIs to use the recommended macros.

### 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. -->
This fix addresses PR feedback from microsoft#25454 .

---------

Co-authored-by: Aditya Rastogi <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

5 participants