Skip to content

Conversation

@skottmckay
Copy link
Contributor

Description

Update some new API functions to directly return the value if they're simple getters.

Motivation and Context

Simplify API usage.

snnn
snnn previously approved these changes Jul 20, 2025
@snnn snnn dismissed their stale review July 20, 2025 00:29

Has build failures

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.

@jywu-msft jywu-msft merged commit 024ae9e into main Jul 22, 2025
128 of 133 checks passed
@jywu-msft jywu-msft deleted the skottmckay/SimplifyApiForGetters branch July 22, 2025 16:33
@snnn
Copy link
Contributor

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
… simple getters. (microsoft#25459)

### Description
<!-- Describe your changes. -->
Update some new API functions to directly return the value if they're
simple getters.

### 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. -->
Simplify API usage.
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.

4 participants