Skip to content

Support override of free dimensions by name#3529

Closed
jeffbloo wants to merge 5 commits into
masterfrom
user/jeffbloo/FreeDimOverrideByName
Closed

Support override of free dimensions by name#3529
jeffbloo wants to merge 5 commits into
masterfrom
user/jeffbloo/FreeDimOverrideByName

Conversation

@jeffbloo
Copy link
Copy Markdown
Contributor

This enables sizes of free dimensions to be specified by name while a session is created, in addition to denotation.

The existing API which is based on denotation is preserved with binary compatibility (though renamed to AddFreeDimensionOverrideByDenotation) and is still used by Winml. A new API named AddFreeDimensionOverrideByName) is added.

The new API enables models which contain free width and height dimensions, for example, to be fully optimized during creation of the session.

@jeffbloo jeffbloo requested a review from pranavsharma April 14, 2020 23:55
@jeffbloo jeffbloo requested a review from a team as a code owner April 14, 2020 23:55
Comment thread include/onnxruntime/core/session/onnxruntime_c_api.h Outdated
Comment thread onnxruntime/core/optimizer/free_dim_override_transformer.cc Outdated
@jeffbloo jeffbloo force-pushed the user/jeffbloo/FreeDimOverrideByName branch from ad6259f to a3a8a53 Compare April 15, 2020 23:22
pranavsharma
pranavsharma previously approved these changes Apr 16, 2020
jeffbloo added a commit that referenced this pull request Apr 22, 2020
This merges ORT Github master from to ADO DmlDev, in preparation with merging back to Github for ORT 1.3.

This branch includes changes from the following PRs, which will now be closed:
#3533
#3529
https://microsoft.visualstudio.com/WindowsAI/_git/onnxruntime/pullrequest/4569080

Related work items: #25935271
@jeffbloo
Copy link
Copy Markdown
Contributor Author

This change will be merged instead as part of the DML execution provider payload for ORT 1.3.

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.

2 participants