Skip to content

[rel-1.16.0] Cherry-pick 16940 and 17523 - #17506

Merged
natke merged 4 commits into
rel-1.16.0from
weicwang/cp_r3
Sep 14, 2023
Merged

[rel-1.16.0] Cherry-pick 16940 and 17523#17506
natke merged 4 commits into
rel-1.16.0from
weicwang/cp_r3

Conversation

@Lafi7e

@Lafi7e Lafi7e commented Sep 12, 2023

Copy link
Copy Markdown
Contributor

Cherry-pick 16940 and 17523 for rel-1.16.0.

### Description
Fix float 8 rounding issues discovered in issue #16938 (only CPU
provider).
pranavsharma
pranavsharma previously approved these changes Sep 12, 2023
xadupre
xadupre previously approved these changes Sep 12, 2023
pranavsharma
pranavsharma previously approved these changes Sep 12, 2023
#16715)" (#17523)

This reverts commit bb136f8, then
re-implement it in a different way.
I reverted the original change, then added a version constraint to the
find_package args.

If you still found it picks up wrong gtest version after this change,
you may disable `find_package` by setting
'FETCHCONTENT_TRY_FIND_PACKAGE_MODE' to NEVER. For example, the latest
gtest version is 1.14.0. If at a later time Google releases a new
version of gtest and that one is incompatible with the ONNX Runtime
source code you get today and your dev environment already installed the
new version and you do not want to create a new clean build environment
that is without the package, you can add `--cmake_extra_defines
FETCHCONTENT_TRY_FIND_PACKAGE_MODE=NEVER` to your build command to solve
the problem.
@Lafi7e
Lafi7e dismissed stale reviews from pranavsharma and xadupre via 4daa97b September 13, 2023 05:47
@Lafi7e Lafi7e changed the title [rel-1.16.0] Cherry-pick 16940 [rel-1.16.0] Cherry-pick 16940 and 17523 Sep 13, 2023
snnn
snnn previously approved these changes Sep 13, 2023
@snnn
snnn dismissed their stale review September 13, 2023 14:19

Gtest version is wrong

Comment thread cmake/external/onnxruntime_external_deps.cmake Outdated
@natke
natke merged commit 06ea28b into rel-1.16.0 Sep 14, 2023
@natke
natke deleted the weicwang/cp_r3 branch September 14, 2023 17:46
@snnn snnn mentioned this pull request Oct 30, 2023
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.

5 participants