Skip to content

Conversation

@wcy123
Copy link
Contributor

@wcy123 wcy123 commented May 17, 2025

Description

convert all session configs, i.e. key-value pairs into provider options, the key prefixed with ort_session_config.

Motivation and Context

#24445 has a bug when Ort::CompileModel is used, not all session config are passed to VITISAI EP backend.
It is because that the session_option which holds a reference to VitisiAIExectuionProviderFactory is not as same as the session_option used for Ort::CompileModel. Ort::CompileModel create another session_option behind scene.

The symptom of this bug is that only the session configs in the first SessionOptions object is passed to VitisiAIExectuionProviderFactory and session configs in the second SessionOptions are not, so that VITISAI EP backend sometimes assumes that ep.cache_context is not enabled, and then ep context cache model is not created properly.

@wcy123 wcy123 changed the title pass session configs to vitisai ep [VITISAI] pass session configs to vitisai ep May 17, 2025
@wcy123 wcy123 changed the title [VITISAI] pass session configs to vitisai ep [VITISAI] pass all session configs to vitisai ep for Ort::CompileModel flow May 17, 2025
@HectorSVC
Copy link
Contributor

/azp run Linux QNN CI Pipeline,Win_TRT_Minimal_CUDA_Test_CI,Windows ARM64 QNN CI Pipeline,Windows GPU Doc Gen CI Pipeline,Windows x64 QNN CI Pipeline

@azure-pipelines
Copy link

Azure Pipelines successfully started running 5 pipeline(s).

@tianleiwu
Copy link
Contributor

@wcy123
Copy link
Contributor Author

wcy123 commented May 19, 2025

@wcy123, please follow https://github.com/microsoft/onnxruntime/blob/main/docs/Coding_Conventions_and_Standards.md#linting to format the code.

thank for your comment. linter is done.

@HectorSVC
Copy link
Contributor

/azp run Linux QNN CI Pipeline,Win_TRT_Minimal_CUDA_Test_CI,Windows ARM64 QNN CI Pipeline,Windows GPU Doc Gen CI Pipeline,Windows x64 QNN CI Pipeline

@azure-pipelines
Copy link

Azure Pipelines successfully started running 5 pipeline(s).

@HectorSVC HectorSVC added the ep:VitisAI issues related to Vitis AI execution provider label May 19, 2025
@HectorSVC HectorSVC merged commit 8eaeec3 into microsoft:main May 19, 2025
81 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ep:VitisAI issues related to Vitis AI execution provider

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants