Skip to content

fix: handle setBindingDimensions return value in TensorRT EP#11929

Merged
jywu-msft merged 1 commit intomicrosoft:masterfrom
senysenyseny16:fix/tensorrt_dim_bindings
Jun 21, 2022
Merged

fix: handle setBindingDimensions return value in TensorRT EP#11929
jywu-msft merged 1 commit intomicrosoft:masterfrom
senysenyseny16:fix/tensorrt_dim_bindings

Conversation

@senysenyseny16
Copy link
Copy Markdown
Contributor

Handle status of setBindingDimensions (link) in TensorRT EP.

Motivation and Context
Without checking the result of setBindingDimensions TensorRT may load cached engine for model with different input shape, for example it can load engine generated for model with dynamic shapes for model with fixed shape.

@jywu-msft jywu-msft requested a review from stevenlix June 21, 2022 16:13
Comment thread onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc Outdated
@jywu-msft
Copy link
Copy Markdown
Member

/azp run Linux GPU TensorRT CI Pipeline, Windows GPU TensorRT CI Pipeline

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 2 pipeline(s).

@jywu-msft
Copy link
Copy Markdown
Member

/azp run onnxruntime-binary-size-checks-ci-pipeline, onnxruntime-python-checks-ci-pipeline, orttraining-linux-ci-pipeline, orttraining-linux-gpu-ci-pipeline, orttraining-ortmodule-distributed

@jywu-msft
Copy link
Copy Markdown
Member

/azp run Linux CPU CI Pipeline, Linux CPU Minimal Build E2E CI Pipeline, Linux GPU CI Pipeline, Linux Nuphar CI Pipeline, MacOS CI Pipeline, ONNX Runtime Web CI Pipeline, Windows CPU CI Pipeline, Windows GPU CI Pipeline

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 5 pipeline(s).

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 8 pipeline(s).

@jywu-msft
Copy link
Copy Markdown
Member

/azp run Linux OpenVINO CI Pipeline

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@jywu-msft jywu-msft merged commit 8c8a781 into microsoft:master Jun 21, 2022
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.

3 participants