-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[TensorRT EP] update oss parser to latest #23710
Conversation
The TRT CI has building issue. |
onnx-tensorrt introduced a dependency plugin.h, which is available by tarball/zip installation (under |
this is a bit confusing. if we publish packages built against TRT 10.8 but stay with10.7 parser? |
Why previously when our CI was running with TRT 10.6 built-in parser didn't complain about this plugin.h depedency? |
This dependency is only needed by oss parser. CIs that use Bulit-in parser have no issue even though there's no plugin.h |
You mention "Updating the logic to make sure builtin parser users are not blocked", it should be oss parser, right? |
I didn't mention that clearly. |
### Description <!-- Describe your changes. --> * Update oss parser version to latest commit of 10.8-GA branch ### 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. --> * Action needed to adapt latest onnx-tensorrt 10.8-GA branch to fix scatterND attribute issue and `plugin.h` not found issue
### Description <!-- Describe your changes. --> * Update oss parser version to latest commit of 10.8-GA branch ### 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. --> * Action needed to adapt latest onnx-tensorrt 10.8-GA branch to fix scatterND attribute issue and `plugin.h` not found issue
Cherry-pick the following changes into [rel-1.21.0](https://github.com/microsoft/onnxruntime/tree/rel-1.21.0). - (#23791) - (#23710) - (#23789) - (#23829) --------- Co-authored-by: Edward Chen <18449977+edgchen1@users.noreply.github.com> Co-authored-by: Yifan Li <109183385+yf711@users.noreply.github.com> Co-authored-by: Ankit Maheshkar <ankit.maheshkar@intel.com> Co-authored-by: n1harika <niharika.sathish@intel.com> Co-authored-by: Changming Sun <chasun@microsoft.com>
Description
Motivation and Context
plugin.h
not found issue