Skip to content

build.sh: don't require user to set --use_full_protobuf with --use_tensorrt option. we can set it implicitly. - #780

Merged
snnn merged 2 commits into
masterfrom
jywu_trt_protobuf
Apr 6, 2019
Merged

build.sh: don't require user to set --use_full_protobuf with --use_tensorrt option. we can set it implicitly.#780
snnn merged 2 commits into
masterfrom
jywu_trt_protobuf

Conversation

@jywu-mysoft

Copy link
Copy Markdown
Contributor

reduce the number of build options required to set for tensorrt build.
--use_full_protobuf option be set implicitly when --use_tensorrt is set.
fix up some other portions of the BUILD instructions.

@jywu-mysoft
jywu-mysoft requested a review from a team as a code owner April 5, 2019 21:47
@snnn
snnn merged commit 571291c into master Apr 6, 2019
@snnn
snnn deleted the jywu_trt_protobuf branch April 6, 2019 17:12
jnagi-intel pushed a commit to jnagi-intel/onnxruntime that referenced this pull request Jan 5, 2026
Sync with Microsoft ONNX Runtime - [18/08/2025]
yuslepukhin pushed a commit that referenced this pull request Mar 17, 2026
## Describe your changes
 Simplify perf tuning logging output

Before:

![image](https://github.com/microsoft/Olive/assets/13343117/287dd346-2c42-4dd2-ae99-11c1278e3cd9)

After:

![image](https://github.com/microsoft/Olive/assets/13343117/2a44d3ad-3648-4632-aaf5-0cd4e4d9828c)


## Checklist before requesting a review
- [ ] Add unit tests for this change.
- [ ] Make sure all tests can pass.
- [ ] Update documents if necessary.
- [ ] Lint and apply fixes to your code by running `lintrunner -a`
- [ ] Is this a user-facing change? If yes, give a description of this
change to be included in the release notes.

## (Optional) Issue link
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