Skip to content
This repository has been archived by the owner on Oct 13, 2021. It is now read-only.

Support onnx 1.7 and ort 1.3 in UT and nightly build. #496

Merged
merged 39 commits into from
May 22, 2020

Conversation

jiafatom
Copy link
Collaborator

@jiafatom jiafatom commented May 20, 2020

Support onnx 1.7 and ort 1.3 in UT and nightly build.

@jiafatom jiafatom requested a review from wenbingl May 20, 2020 22:42
@jiafatom jiafatom changed the title Add opset 12 UT, upgrade to ort 1.3 in UT Add opset 12 UT, upgrade to ort 1.3 in UT, fix shrink_mask May 21, 2020
keras2onnx/_builtin.py Outdated Show resolved Hide resolved
@jiafatom jiafatom changed the title Add opset 12 UT, upgrade to ort 1.3 in UT, fix shrink_mask Add opset 12 UT, upgrade to ort 1.3 in UT May 21, 2020
@jiafatom jiafatom changed the title Add opset 12 UT, upgrade to ort 1.3 in UT Support onnx 1.7 and ort 1.3 in UT and nightly build. May 21, 2020
.azure-pipelines/linux-conda-CI-tf-keras.yml Outdated Show resolved Hide resolved
.azure-pipelines/linux-conda-CI.yml Outdated Show resolved Hide resolved
keras2onnx/_builtin.py Show resolved Hide resolved
@lgtm-com
Copy link

lgtm-com bot commented May 22, 2020

This pull request fixes 1 alert when merging b220e93 into bb5c193 - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@lgtm-com
Copy link

lgtm-com bot commented May 22, 2020

This pull request fixes 1 alert when merging 9833722 into bb5c193 - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@lgtm-com
Copy link

lgtm-com bot commented May 22, 2020

This pull request fixes 1 alert when merging c88fa54 into bb5c193 - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@lgtm-com
Copy link

lgtm-com bot commented May 22, 2020

This pull request fixes 1 alert when merging 2c11347 into bb5c193 - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

.azure-pipelines/linux-conda-CI.yml Show resolved Hide resolved
@@ -28,13 +28,21 @@ jobs:
TENSORFLOW_PATH: tensorflow==1.15.0
INSTALL_ORT: pip install onnxruntime==1.1.1

Python37-tf200:
# UT for standalone keras need tensorflow <= 2.0.0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove it too.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure

Python37-tf200:
python.version: '3.7'
ONNX_PATH: onnx==1.6.0
ONNX_PATH: onnx==1.7.0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1.6?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@lgtm-com
Copy link

lgtm-com bot commented May 22, 2020

This pull request fixes 1 alert when merging d6276ab into bb5c193 - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@jiafatom jiafatom merged commit cee609f into onnx:master May 22, 2020
@jiafatom jiafatom deleted the op17_test_1 branch May 22, 2020 23:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants